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

Expose HTTP response headers in websocket upgrade request #6221

Closed
2 tasks
esauerbo opened this issue Jun 24, 2024 · 2 comments
Closed
2 tasks

Expose HTTP response headers in websocket upgrade request #6221

esauerbo opened this issue Jun 24, 2024 · 2 comments
Assignees
Labels
closed-for-staleness feature-request New feature or enhancement. May require GitHub community feedback. wontfix We have determined that we will not resolve the issue.

Comments

@esauerbo
Copy link

Describe the feature

It would be helpful to have access to the response headers in the initial http upgrade request for a websocket connection. We are specifically looking to access the date. From the below screenshots you can see the headers in the network tab but not when logging the response.

Screenshot 2024-06-13 at 10 04 49 AM
Screenshot 2024-06-13 at 10 05 01 AM

Use Case

The Amplify UI Face Liveness Detector provides a UI component for Amazon Rekognition Face Liveness using the client-rekognitionstreaming package. Some customers are getting Invalid Signature Exceptions due to clock skew issues.

We want to fix this by catching these exceptions and attempting to restart the session with a systemClockOffset that's the difference between device and server time. This has been implemented on the android liveness component, but currently isn't possible in web because we cannot access the http upgrade headers.

Proposed Solution

No response

Other Information

No response

Acknowledgements

  • I may be able to implement this feature request
  • This feature might incur a breaking change

SDK version used

V3

Environment details (OS name and version, etc.)

any

@esauerbo esauerbo added feature-request New feature or enhancement. May require GitHub community feedback. needs-triage This issue or PR still needs to be triaged. labels Jun 24, 2024
@kuhe kuhe self-assigned this Jun 25, 2024
@kuhe kuhe added p1 This is a high priority issue wontfix We have determined that we will not resolve the issue. and removed needs-triage This issue or PR still needs to be triaged. p1 This is a high priority issue labels Jun 25, 2024
@kuhe
Copy link
Contributor

kuhe commented Jun 26, 2024

there doesn't seem to be a way to retrieve upgrade response headers in a browser using the WebSocket class.

@kuhe kuhe added the closing-soon This issue will automatically close in 4 days unless further comments are made. label Jun 26, 2024
@github-actions github-actions bot added closed-for-staleness and removed closing-soon This issue will automatically close in 4 days unless further comments are made. labels Jul 1, 2024
@github-actions github-actions bot closed this as completed Jul 1, 2024
Copy link

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs and link to relevant comments in this thread.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 15, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
closed-for-staleness feature-request New feature or enhancement. May require GitHub community feedback. wontfix We have determined that we will not resolve the issue.
Projects
None yet
Development

No branches or pull requests

2 participants