Add ability to inject extra claims and xsrfToken by lwcolton · Pull Request #248 · stormpath/stormpath-sdk-python · GitHub
Skip to content

Add ability to inject extra claims and xsrfToken - #248

Open
lwcolton wants to merge 6 commits into
stormpath:masterfrom
HurricaneLabs:add-extra-claims-csrf
Open

Add ability to inject extra claims and xsrfToken#248
lwcolton wants to merge 6 commits into
stormpath:masterfrom
HurricaneLabs:add-extra-claims-csrf

Conversation

@lwcolton

@lwcolton lwcolton commented Apr 2, 2016

Copy link
Copy Markdown

JWT generation functions were moved into the JWTFactory class.
The JWTRequestAuthenticator class was added,and was made the
parent class of ApiRequestAuthenticator, OAuthRequestAuthenticator,
and OAuthClientCredentialsRequestAuthenticator. These three
authenticator classes now allow specifying arbitrary key-value
pairs to add in the JWT claims. The also now have support for
auto-generating a uuid4 to be used as a CSRF token in the JWT,
as described in the "Cookies" section of
https://stormpath.com/blog/where-to-store-your-jwts-cookies-vs-html5-web-storage/

Colton Leekley-Winslow added 6 commits April 2, 2016 17:56
JWT generation functions were moved into the JWTFactory class.
The JWTRequestAuthenticator class was added,and was made the
parent class of ApiRequestAuthenticator, OAuthRequestAuthenticator,
and OAuthClientCredentialsRequestAuthenticator.  These three
authenticator classes now allow specifying arbitrary key-value
pairs to add in the JWT claims.  The also now have support for
auto-generating a uuid4 to be used as a CSRF token in the JWT,
as described in the "Cookies" section of
https://stormpath.com/blog/where-to-store-your-jwts-cookies-vs-html5-web-storage/
@lwcolton

lwcolton commented Apr 7, 2016

Copy link
Copy Markdown
Author

@rdegges

rdegges commented Apr 7, 2016

Copy link
Copy Markdown
Contributor

Thanks for the PR! Going to review this as soon as I have a chance. I've been out on vacation, and am heading to a conference in NYC for a few days. Sorry for the delay :(

@rdegges

rdegges commented Jun 13, 2016

Copy link
Copy Markdown
Contributor

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