search

awslabs / aws-mobile-appsync-sdk-js

JavaScript library files for Offline, Sync, Sigv4. includes support for React Native
Apache License 2.0
917 stars 265 forks source link

fix: headerBasedAuth not updating the headers with new values #667

Open chrisharrison opened 2 years ago

chrisharrison commented 2 years ago

Issue #, if available:

666

Description of changes:

Allows the headerBasedAuth function to actually update the headers with the new value.

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

chrisharrison commented 2 years ago

This fixes #666

chrisharrison commented 2 years ago

This should be applied as a hotfix to aws-appsync-auth-link version 2.0.5

ronnyroeller commented 10 months ago

The Amplify docs still advises to use this repo when using Apollo Client: https://docs.amplify.aws/lib/graphqlapi/upgrade-guide/q/platform/js/#option-2-upgrade-to-apollo-client-with-appsync-links

Wouldn't it make sense to merged this PR then? Or is there by now an alternative advised approach of how to use Apollo Client with AppSync?