Skip to content
This repository has been 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
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

phxql
Copy link

@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).

session.go Outdated Show resolved Hide resolved
session.go Show resolved Hide resolved
session.go Show resolved Hide resolved
session.go Show resolved Hide resolved
session_test.go Show resolved Hide resolved
session.go Show resolved Hide resolved
session.go Outdated Show resolved Hide resolved
@phxql phxql force-pushed the feature/ignore-authorization-header branch from 7c14c0f to e39e799 Compare January 20, 2020 15:43
session.go Outdated Show resolved Hide resolved
session.go Show resolved Hide resolved
session.go Show resolved Hide resolved
@phxql phxql force-pushed the feature/ignore-authorization-header branch 4 times, most recently from 0398b04 to 06a47a2 Compare January 21, 2020 07:47
@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
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