Skip to content
This repository has been archived by the owner on Mar 12, 2020. It is now read-only.

Update github.com/dgrijalva/jwt-go to 3.2.0 #27

Merged
merged 1 commit into from
Jun 19, 2019

Conversation

guillaumerose
Copy link
Contributor

@guillaumerose guillaumerose commented Jun 14, 2019

closes #25

In Docker Desktop, we vendor labstack/echo which use the latest version of jwt-go (which is already more than 1 year old).

I had to update the code to match the new API. All tests are green.

Signed-off-by: Guillaume Rose <guillaume.rose@docker.com>
@thaJeztah
Copy link
Member

We'll have to update docker/cli#1936 once this is merged, to get the changes in

Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - ran tests locally

@thaJeztah
Copy link
Member

ping @mason-fish @dhiltgen PTAL

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants