Skip to content
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

feat: upgrade to graphqlws #1424

Closed
wants to merge 8 commits into from
Closed

feat: upgrade to graphqlws #1424

wants to merge 8 commits into from

Conversation

nicolasburtey
Copy link
Member

@nicolasburtey nicolasburtey commented Jul 10, 2022

fix #1365, plus some security consideration using the old library: enisdenjo/graphql-ws#3

I ran into issues when I was importing both subscriptions-transport-ws and graphql-ws at the same time. instead of trying to understand the root of this, I decided to duplicated graphql-server and dynamically, at runtime, one or the other websocket library would be used.

by default the old library would be used - so that we can enable the use of the new library in another PR in the charts/deployment repo.

I recommend that we use a new url for BBW to do the migration from the mobile app. we could use api.bbw.sv to replace the current url. api.bbw.sv would have the graphql-ws library. mainnet.graphql.galoy.io would have the subscriptions-transport-ws library.

@nicolasburtey nicolasburtey changed the title Update graphqlws feat: upgrade to graphqlws Jul 10, 2022
@nicolasburtey nicolasburtey force-pushed the update-graphqlws branch 6 times, most recently from d905bbc to d3246a3 Compare July 17, 2022 14:20
@nicolasburtey nicolasburtey force-pushed the update-graphqlws branch 6 times, most recently from 24d49c5 to 1cd6d85 Compare July 24, 2022 16:31
@nicolasburtey nicolasburtey force-pushed the update-graphqlws branch 2 times, most recently from 749d93c to 99e4509 Compare August 1, 2022 09:30
@nicolasburtey nicolasburtey marked this pull request as ready for review August 8, 2022 12:55
may need more work for cross compatibility with older transport library that is being used in the mobile repo
otherwise there seems to be a critical js error happening, at least on the CI server.

probably some minor package update that has a regression/incompatibility
@nicolasburtey
Copy link
Member Author

legacy server (when no env variable is set) --> api.mainnet.galoy.io
"new" server (when NEW_SERVER is set) --> api.bbw.sv

@nicolasburtey
Copy link
Member Author

migrating in favor of #1424 (a squash to simplify the rebase)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

switch to new graphql subscription protocol
1 participant