-
Notifications
You must be signed in to change notification settings - Fork 12
Update custom domain publishing approach #3917
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
21 commits
Select commit
Hold shift + click to select a range
828469c
Change override to use permissions over user
backspace d2e64bf
Add teal paper domain
backspace c0aec22
Merge remote-tracking branch 'origin/main' into server/override-publi…
backspace d240409
Add UI for override
backspace f2c65a2
Fix import order
backspace e30523e
Change mapping to use base realm URL
backspace 9e9655e
Fix lint errors
backspace 10d8835
Add environment variable for more overrides
backspace 9696704
Add variable to host config
backspace 43c21de
Fix types in parsing
backspace 1fa17f1
Extract handling into contained helper
backspace 54da3b0
Fix lint error
backspace f49235e
Update colours of override warning
backspace 4be20c6
Add lint autofix
backspace 9aa4efc
Merge branch 'main' into server/override-publish-cs-10117
backspace fab1e03
Update handling on both sides
backspace b18f115
Add formatting autofix
backspace b41e9b9
Fix type error
backspace 2237359
Merge remote-tracking branch 'origin/main' into server/override-publi…
backspace 13975e5
Add formatting autofixes
backspace a522c04
Move overrides entirely into environment variables
backspace File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The override value is fed directly into
URL.host, but overrides can come from env JSON and the server-side parser explicitly accepts full URLs (e.g.https://whitepaper.boxel.ai). If the override includes a scheme,overriddenURL.host = overrideDomainproduces a malformed URL likehttps://https/, which then shows a broken URL in the modal and causes publish requests to fail for those overrides. Consider parsing/normalizing the override string to hostname/port (or stripping scheme) before assigning it tohost.Useful? React with 👍 / 👎.