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

SignedString is an empty string #8

Open
bonifaido opened this issue Feb 7, 2018 · 2 comments
Open

SignedString is an empty string #8

bonifaido opened this issue Feb 7, 2018 · 2 comments

Comments

@bonifaido
Copy link

I think there is an issue with the token signing, which is that the value of SignedString is never set (so it is the default empty string) but it is used for signing tokens:

signedToken, _ := token.SignedString([]byte(sessionStorer.SignedString))

@sergolius
Copy link

Default SessionStorer obviously has omitted SignedString, it's secret of a session.

auth/auth.go

Lines 88 to 94 in 709ba0c

if config.SessionStorer == nil {
config.SessionStorer = &SessionStorer{
SessionName: "_auth_session",
SessionManager: manager.SessionManager,
SigningMethod: jwt.SigningMethodHS256,
}
}

You have to define it on your own.

@ghost
Copy link

ghost commented Feb 29, 2020

I created a fork of qor where I committed several fixes/pull requests to make it work. Available at, https://github.com/qorpress/qorpress-auth-example

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

2 participants