Skip to content
This repository has been archived by the owner on Oct 4, 2023. It is now read-only.

feat: use new dedicated websocket server #220

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

ntheile
Copy link
Contributor

@ntheile ntheile commented Jul 3, 2023

No description provided.

@ntheile
Copy link
Contributor Author

ntheile commented Jul 4, 2023

@nicolasburtey / @krtk6160 ,

The web wallet code in the PR works with the dedicated websocket server locally and in my local dev charts environment but not in staging when I call authenticated subscriptions using cookies (unauthenticated still works). I am not sure why?

To reproduce you can pull this code down and set these .envrc settings:

export NETWORK="signet"
export GRAPHQL_URL="https://api.staging.galoy.io/graphql"
export GRAPHQL_SUBSCRIPTION_URL="wss://ws.staging.galoy.io/graphql"
export GALOY_AUTH_ENDPOINT="https://api.staging.galoy.io/auth"

I get this error below, when testing an authenticated subscription to myUpdates:

{message: "Not Authenticated for subscription", locations: [{line: 2, column: 3}], path: ["myUpdates"],…}

Relevant honeycomb traces. https://ui.honeycomb.io/galoy/datasets/galoy-staging/result/5w7PYmskvfA?tab=raw

Any ideas? I wonder if oathkeeper or nginx is not passing the cookies properly

@ntheile ntheile force-pushed the feat--use-new-dedicated-websocket-server branch from 83875b3 to 505653d Compare August 1, 2023 17:56
galoybot pushed a commit that referenced this pull request Aug 25, 2023
* feat: search by email

* chore: fix email verified

---------

Co-authored-by: Nicolas Burtey <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant