Skip to content
This repository was archived by the owner on Dec 7, 2020. It is now read-only.

KEYCLOAK-12731 Add option 'UseAuthorizationHeader' to disable reading token from Authorization header#499

Open
phxql wants to merge 1 commit intolouketo:masterfrom
phxql:feature/ignore-authorization-header
Open

KEYCLOAK-12731 Add option 'UseAuthorizationHeader' to disable reading token from Authorization header#499
phxql wants to merge 1 commit intolouketo:masterfrom
phxql:feature/ignore-authorization-header

Conversation

@phxql
Copy link
Copy Markdown

@phxql phxql commented Jan 20, 2020

This makes it possible to protect applications with the gatekeeper which use the Authorization header with their own JWT token. Default of this value is true (backward compatible).

Comment thread session.go Outdated
Comment thread session.go
Comment thread session.go
Comment thread session.go
Comment thread session_test.go
Comment thread session.go
Comment thread session.go Outdated
@phxql phxql force-pushed the feature/ignore-authorization-header branch from 7c14c0f to e39e799 Compare January 20, 2020 15:43
Comment thread session.go Outdated
Comment thread session.go
Comment thread session.go
@phxql phxql force-pushed the feature/ignore-authorization-header branch 4 times, most recently from 0398b04 to 06a47a2 Compare January 21, 2020 07:47
Comment thread session.go
@phxql phxql force-pushed the feature/ignore-authorization-header branch 2 times, most recently from d69f646 to 74fee07 Compare January 21, 2020 07:52
… token from Authorization header

This makes it possible to protect applications with the gatekeeper which use the Authorization header with their own JWT token. Default of this value is true (backward compatible).
@phxql phxql force-pushed the feature/ignore-authorization-header branch from 74fee07 to 2961222 Compare January 21, 2020 08:16
@phxql phxql requested a review from abstractj January 22, 2020 07:42
@abstractj abstractj self-assigned this Jan 22, 2020
@phxql
Copy link
Copy Markdown
Author

phxql commented Jan 29, 2020

Hi, do you need any information on this PR? Do i need to do anything apart from wait? :)

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