Skip to content

Comments

Fix: don't JSON-encode JWT and update content type#121

Merged
m-hulbert merged 2 commits intojwt-authentication-nodejsfrom
fix/jwt-auth-remove-jsonify
Oct 3, 2024
Merged

Fix: don't JSON-encode JWT and update content type#121
m-hulbert merged 2 commits intojwt-authentication-nodejsfrom
fix/jwt-auth-remove-jsonify

Conversation

@m-hulbert
Copy link

This PR updates the JWT Auth tutorial code:

  • There isn't a need to JSON-encode a JWT
  • The content type shouldn't be application/json
  • Updated minimum ably-js version to v2+

@m-hulbert m-hulbert requested a review from GregHolmes October 2, 2024 20:52
@m-hulbert m-hulbert self-assigned this Oct 2, 2024
@m-hulbert m-hulbert force-pushed the fix/jwt-auth-remove-jsonify branch from e5630a4 to 65a5265 Compare October 3, 2024 10:53
@m-hulbert m-hulbert force-pushed the fix/jwt-auth-remove-jsonify branch from 65a5265 to 26e7a85 Compare October 3, 2024 10:58
@m-hulbert m-hulbert merged commit 6d2e15a into jwt-authentication-nodejs Oct 3, 2024
@m-hulbert m-hulbert deleted the fix/jwt-auth-remove-jsonify branch October 3, 2024 11:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants