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.

Sign up for GitHub

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

Jump to bottom

auth/jwt: prevent concurrent reads and writes on MapClaims #564

Merged
merged 5 commits into from
Jul 17, 2017
Merged

auth/jwt: prevent concurrent reads and writes on MapClaims #564

merged 5 commits into from
Jul 17, 2017

Commits on Jul 1, 2017

  1. auth/jwt: MapClaims: failing test

    @peterbourgon
    peterbourgon committed Jul 1, 2017
    Configuration menu
    Copy the full SHA
    c4aa78e View commit details
    Browse the repository at this point in the history

Commits on Jul 11, 2017

  1. auth/jwt: MapClaims: passing 

    add claimsFactory type
make NewParser take a claimsFactory instead of an instance of jwt.Claims
use claimsFactory to create a jwt.Claims to pass in to jwt.ParseWithClaims
update NewParser calls in tests to take a claimsFactory instead of a jwt.Claims instance
    @litriv
    litriv committed Jul 11, 2017
    Configuration menu
    Copy the full SHA
    fe8e143 View commit details
    Browse the repository at this point in the history

Commits on Jul 12, 2017

  1. auth/jwt: MapClaims: export ClaimsFactory and provide implementations… 

    … for Map and Standard claims factories
    @litriv
    litriv committed Jul 12, 2017
    Configuration menu
    Copy the full SHA
    44bb404 View commit details
    Browse the repository at this point in the history

Commits on Jul 13, 2017

  1. auth/jwt: MapClaims: add doc comment for ClaimsFactory type

    @litriv
    litriv committed Jul 13, 2017
    Configuration menu
    Copy the full SHA
    37eab0a View commit details
    Browse the repository at this point in the history

  2. Merge pull request #568 from litriv/issue-562 

    auth/jwt: MapClaims: passing
    @peterbourgon
    peterbourgon authored Jul 13, 2017
    Configuration menu
    Copy the full SHA
    b42a850 View commit details
    Browse the repository at this point in the history