Skip to content

fix: inject auth info to appsync handshake headers #22

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 1 commit into from
Sep 20, 2024

Conversation

5d
Copy link
Contributor

@5d 5d commented Sep 20, 2024

Issue #, if available:

Description of changes:

  • inject auth info to AppSync realtime websocket handshake request headers

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@5d 5d force-pushed the 5d/appsync-auth-headers branch from cbeff59 to f4b723c Compare September 20, 2024 18:00
@5d 5d force-pushed the 5d/appsync-auth-headers branch from f4b723c to f8fa4f6 Compare September 20, 2024 18:19
@5d 5d marked this pull request as ready for review September 20, 2024 18:49
@5d 5d requested a review from a team as a code owner September 20, 2024 18:49
Copy link
Contributor

@lawmicha lawmicha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you make sure the integration tests are passing with these changes?

@5d
Copy link
Contributor Author

5d commented Sep 20, 2024

Can you make sure the integration tests are passing with these changes?

I've manually confirmed that all integration test cases have passed.

@5d 5d merged commit 3f98d81 into main Sep 20, 2024
8 checks passed
@5d 5d deleted the 5d/appsync-auth-headers branch September 20, 2024 20:26
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.

2 participants