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

Fix typo in jwt package #1070

Merged
merged 5 commits into from
Jun 19, 2021
Merged

Fix typo in jwt package #1070

merged 5 commits into from
Jun 19, 2021

Conversation

amidam
Copy link
Contributor

@amidam amidam commented Mar 19, 2021 

* change JWTTokenContextKey to JWTContextKey

* revise errors as well as comments

Fix typo in jwt package
ce02833 
    * change JWTTokenContextKey to JWTContextKey

    * revise errors as well as comments
@amidam
Copy link
Contributor Author

amidam commented Mar 19, 2021

fixes #1071

peterbourgon
peterbourgon requested changes Mar 20, 2021
View reviewed changes
Revision of typo fixing in jwt package
ab85124 
	* revert the API identifier to its previous value

        * add JWTTokenContextKey side by side of JWTContextKey for historical compatibility
@amidam amidam requested a review from peterbourgon March 20, 2021 14:19
@peterbourgon
Copy link
Member

Can you please leave the old identifiers in place, and mark them as deprecated?

Amid added 2 commits June 2, 2021 21:54
Fixed a typo in JWT package (go-kit#1071)
2279904 
	* define JWTContextKey as a new constant

        * mark JWTTokenContextKey as a deprecated constant

        * revise corresponding error messages
Merge branch 'master' of https://github.com/amidam/kit
f0a32e2 
# Conflicts:
#	auth/jwt/middleware.go
@amidam
Copy link
Contributor Author

amidam commented Jun 2, 2021

Can you please leave the old identifiers in place, and mark them as deprecated?

I have done it already. I added a new constant called JWTContextKey and marked JWTTokenContextKey as a deprecated constant.

@amidam
Copy link
Contributor Author

amidam commented Jun 2, 2021

Can you please leave the old identifiers in place, and mark them as deprecated?

I have done it already. I added a new constant called JWTContextKey and marked JWTTokenContextKey as a deprecated constant.

Still they both have the same value

peterbourgon
peterbourgon requested changes Jun 2, 2021
View reviewed changes
Copy link
Member

@peterbourgon peterbourgon left a comment

Choose a reason for hiding this comment

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

One minor thing then 👍

Fixed a typo in JWT package (go-kit#1071)
00d2948 
	* define JWTContextKey as a new constant

        * mark JWTTokenContextKey as a deprecated constant

        * revise corresponding error messages
@amidam
Copy link
Contributor Author

amidam commented Jun 2, 2021

One minor thing then +1

I have done it.

@amidam amidam requested a review from peterbourgon June 2, 2021 22:36
sagikazarmark
sagikazarmark approved these changes Jun 19, 2021
View reviewed changes
Copy link
Contributor

@sagikazarmark sagikazarmark left a comment

Choose a reason for hiding this comment

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

LGTM

@sagikazarmark sagikazarmark mentioned this pull request Jun 19, 2021
Closed
@peterbourgon peterbourgon merged commit a5c7103 into go-kit:master Jun 19, 2021
@sagikazarmark sagikazarmark added this to the v0.11.0 milestone Jun 19, 2021
@sagikazarmark sagikazarmark mentioned this pull request Jun 19, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sagikazarmark sagikazarmark sagikazarmark approved these changes

@peterbourgon peterbourgon Awaiting requested review from peterbourgon

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v0.11.0
Development

Successfully merging this pull request may close these issues.
3 participants
@amidam @peterbourgon @sagikazarmark