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.

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

Support Silhouette (Play framework) #32

Closed
hailg opened this issue Feb 11, 2016 · 2 comments
Closed

Support Silhouette (Play framework) #32

hailg opened this issue Feb 11, 2016 · 2 comments

Comments

@hailg
Copy link

hailg commented Feb 11, 2016

Hi,
I'm using Silhouette with JWT for authorization. In current implementation, they'll read a custom header for the token. The problem is they don't expect "Bearer" as the prefix. Currently, I cannot set header prefix to be empty (from the source code you will add Bearer if I pass in an empty string).

Is it a good idea to remove the default "Bearer"?

@chenkie
Copy link
Contributor

chenkie commented Feb 11, 2016

Ah I didn't consider that. I'll add support for it.

@mikebridge
Copy link

There is some discussion about the token prefix on the Silhouette list here: https://groups.google.com/forum/#!msg/play-silhouette/9gYkHUybzuo/VtRuLwuPHwAJ

A workaround until Silhouette 4 comes out is to set the headerPrefix to a space (not an empty string or null), e.g.:

provide(AuthConfig, {useValue: new AuthConfig({
    headerPrefix: " ",
})})

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

No branches or pull requests

3 participants