docs: update express-jwt auth property name in examples - Githubissues

graphile / crystal

🔮 Graphile's Crystal Monorepo; home to Grafast, PostGraphile, pg-introspection, pg-sql2 and much more!

https://graphile.org/

Other

12.62k stars 571 forks source link

docs: update express-jwt auth property name in examples #2223

Closed harrybuisman closed 3 weeks ago

harrybuisman commented 3 weeks ago

Description

In recent versions of express-jwt the name of the property in the request object where the JWT payload is set defaults to req.auth

This PR also renames the scope claim to better align with the decoded JWT payload example in the documentation proceeding it.

Performance impact

N/A

Security impact

N/A

Checklist

[x] My code matches the project's code style and yarn lint:fix passes.
[ ] I've added tests for the new feature, and yarn test passes.
[ ] I have detailed the new feature in the relevant documentation.
[ ] I have added this feature to 'Pending' in the RELEASE_NOTES.md file (if one exists).
[ ] If this is a breaking change I've explained why.

changeset-bot[bot] commented 3 weeks ago

⚠️ No Changeset found

Latest commit: b135aa91611aa3918de25a99095e29790298beb9

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR