Title: fix(idp): allow CORS for all clients on token endpoint by melvincarvalho · Pull Request #11 · JavaScriptSolidServer/JavaScriptSolidServer · GitHub
Open Graph Title: fix(idp): allow CORS for all clients on token endpoint by melvincarvalho · Pull Request #11 · JavaScriptSolidServer/JavaScriptSolidServer
X Title: fix(idp): allow CORS for all clients on token endpoint by melvincarvalho · Pull Request #11 · JavaScriptSolidServer/JavaScriptSolidServer
Description: Summary Simplify clientBasedCORS to return true for all clients Fixes OIDC login for web apps loaded from CDN (like Mashlib from unpkg.com) Problem When Mashlib is loaded from CDN, the OIDC token exchange fails with: 400 Bad Request - origin not allowed for client The previous code only allowed CORS for clients with tokenEndpointAuthMethod === 'none', but dynamically registered clients default to client_secret_basic. Solution Solid servers are public and should accept token requests from any web app origin, so clientBasedCORS now returns true for all clients. Test plan Clear browser cache/cookies for test server Navigate to a resource with Mashlib enabled Click "Sign In" and complete OIDC login Verify token exchange succeeds (no 400 error) Verify authenticated user can access protected resources Fixes #10
Open Graph Description: Summary Simplify clientBasedCORS to return true for all clients Fixes OIDC login for web apps loaded from CDN (like Mashlib from unpkg.com) Problem When Mashlib is loaded from CDN, the OIDC token...
X Description: Summary Simplify clientBasedCORS to return true for all clients Fixes OIDC login for web apps loaded from CDN (like Mashlib from unpkg.com) Problem When Mashlib is loaded from CDN, the OIDC token...
Opengraph URL: https://github.com/JavaScriptSolidServer/JavaScriptSolidServer/pull/11
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:b6eace5b-c32c-020c-3368-5edc4e7fecf5 |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | DBC4:C83B7:3F8E86:578E63:69774DFE |
| html-safe-nonce | 0f1b1650b59801348c07eb595a1dddaef4b73df77c7a327384edd21427c03ae2 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJEQkM0OkM4M0I3OjNGOEU4Njo1NzhFNjM6Njk3NzRERkUiLCJ2aXNpdG9yX2lkIjoiODMyMjA2NTk3MDM3MzI4MzMyNiIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | ba761d8241ed3a3d2098ae883bcabab2d61a2c9be35239f04d25186b1d38577a |
| hovercard-subject-tag | pull_request:3133777566 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/JavaScriptSolidServer/JavaScriptSolidServer/pull/11/files |
| twitter:image | https://avatars.githubusercontent.com/u/65864?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/65864?s=400&v=4 |
| og:image:alt | Summary Simplify clientBasedCORS to return true for all clients Fixes OIDC login for web apps loaded from CDN (like Mashlib from unpkg.com) Problem When Mashlib is loaded from CDN, the OIDC token... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | 3310064f35a62c06a4024ba37f41c06836f39376a095c2dfd2c4b693c34965be |
| turbo-cache-control | no-preview |
| diff-view | unified |
| go-import | github.com/JavaScriptSolidServer/JavaScriptSolidServer git https://github.com/JavaScriptSolidServer/JavaScriptSolidServer.git |
| octolytics-dimension-user_id | 205442424 |
| octolytics-dimension-user_login | JavaScriptSolidServer |
| octolytics-dimension-repository_id | 958025407 |
| octolytics-dimension-repository_nwo | JavaScriptSolidServer/JavaScriptSolidServer |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 958025407 |
| octolytics-dimension-repository_network_root_nwo | JavaScriptSolidServer/JavaScriptSolidServer |
| turbo-body-classes | logged-out env-production page-responsive full-width |
| disable-turbo | true |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 67d5f8d1d53c3cc4f49fc3bb8029933c3dc219e6 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width