[snack-sdk][website][snackager] Migrate API server hostname#684
Merged
byronkarlen merged 4 commits intoMay 20, 2026
Merged
Conversation
Contributor
Author
This stack of pull requests is managed by Graphite. Learn more about stacking. |
This was referenced May 8, 2026
byronkarlen
changed the title
byronkarlen
changed the title
byronkarlen
force-pushed
the
byron/_website_read_local_expo_auth_cookie_in_development
branch
from
7e05465 to
f7a23fe
Compare
byronkarlen
force-pushed
the
byron/_snack-sdk_website_snackager_migrate_api_server_hostname
branch
from
d6e1968 to
add7f33
Compare
ide
approved these changes
May 9, 2026
byronkarlen
force-pushed
the
byron/_website_read_local_expo_auth_cookie_in_development
branch
from
f7a23fe to
e4b4a3a
Compare
byronkarlen
force-pushed
the
byron/_snack-sdk_website_snackager_migrate_api_server_hostname
branch
from
add7f33 to
a4eba5b
Compare
byronkarlen
changed the base branch from
byron/_website_read_local_expo_auth_cookie_in_development
to
graphite-base/684
byronkarlen
force-pushed
the
byron/_snack-sdk_website_snackager_migrate_api_server_hostname
branch
from
a4eba5b to
87fd82b
Compare
byronkarlen
force-pushed
the
graphite-base/684
branch
from
e4b4a3a to
5fbee17
Compare
byronkarlen
changed the base branch from
graphite-base/684
to
byron/tsconfig_and_eslint_cleanup
byronkarlen
force-pushed
the
byron/tsconfig_and_eslint_cleanup
branch
from
5fbee17 to
12fcfeb
Compare
byronkarlen
force-pushed
the
byron/_snack-sdk_website_snackager_migrate_api_server_hostname
branch
from
87fd82b to
0d0006d
Compare
byronkarlen
force-pushed
the
byron/tsconfig_and_eslint_cleanup
branch
2 times, most recently
from
711ee44 to
07f3029
Compare
byronkarlen
force-pushed
the
byron/_snack-sdk_website_snackager_migrate_api_server_hostname
branch
from
0d0006d to
dd8021d
Compare
byronkarlen
force-pushed
the
byron/tsconfig_and_eslint_cleanup
branch
from
07f3029 to
c81b819
Compare
byronkarlen
force-pushed
the
byron/_snack-sdk_website_snackager_migrate_api_server_hostname
branch
from
dd8021d to
716cdc7
Compare
byCedric
reviewed
May 12, 2026
| ], | ||
| }, | ||
| "url": "https://exp.host/--/api/v2/development-sessions/notify-close?deviceId=1234", | ||
| "url": "https://exp.host/v2/development-sessions/notify-close?deviceId=1234", |
byronkarlen
force-pushed
the
byron/_snack-sdk_website_snackager_migrate_api_server_hostname
branch
from
716cdc7 to
065b7a7
Compare
byronkarlen
force-pushed
the
byron/tsconfig_and_eslint_cleanup
branch
from
c81b819 to
012c054
Compare
byCedric
reviewed
May 14, 2026
| } | ||
| const id = match[2]; | ||
| const response = await fetch(`https://exp.host/--/api/v2/snack/${id}`, { | ||
| const response = await fetch(`https://api.expo.dev/v2/snack/${id}`, { |
Member
There was a problem hiding this comment.
Any reason we don't use ${process.env.API_SERVER_URL}/v2/snack here?
Contributor
Author
There was a problem hiding this comment.
I think this component is meant to always hit the production api server even when API_SERVER_URL is something else during local dev
byronkarlen
changed the base branch from
byron/tsconfig_and_eslint_cleanup
to
graphite-base/684
byronkarlen
force-pushed
the
byron/_snack-sdk_website_snackager_migrate_api_server_hostname
branch
from
065b7a7 to
9a2aa15
Compare
byronkarlen
force-pushed
the
graphite-base/684
branch
from
012c054 to
8808152
Compare
byronkarlen
deleted the
byron/_snack-sdk_website_snackager_migrate_api_server_hostname
branch
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
Why
For cookie-based auth to work, the API server url needs to be updated from exp.host to api.expo.dev (which shares the top-level domain with snack.expo.dev). api.expo.dev accepts requests in a different format than exp.host.
How
snack-sdk
snack-proxies
snackager
website
Test Plan
Used snack.expo.test locally and verified that thing worked as expected, but we don't have e2e test coverage for this.