-
Notifications
You must be signed in to change notification settings - Fork 563
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
Missing second argument for method uuid.Must() #76
Comments
Try to update the package: |
It won't help. I can fix it in oauth2 package but I don't know how to contribute. |
@LyricTian could you do the change in code. This is simple fix. |
You can try to update the package:
I have found no problems with my use here. // NewV4 returns random generated UUID.
func NewV4() (UUID, error) {
return global.NewV4()
} func Must(u UUID, err error) UUID {
if err != nil {
panic(err)
}
return u
} |
@LyricTian I'm trying with go modules and getting the same error. https://github.com/go-oauth2/oauth2/blob/v3.8.12/store/token.go#L46 Those are the latest tags so |
OK,I have removed the package: |
It's much better but I still see one bug When function require 2 parameters we cannot pass just one even when it can be nil |
OK.You try again: |
It is the other bug Example usage: NewRedisStore returns different than NewTokenStore in redis.v1 |
you can try |
@LyricTian Thanks for the fixes. Could we get a bugfix release tagged for go modules? |
Yes, what do I need to do? |
I bumped my project to |
so this is done and could be closed? |
Example from generates/access.go
access = base64.URLEncoding.EncodeToString(uuid.NewV3(uuid.Must(uuid.NewV4()), buf.String()).Bytes())
Fixed code:
access = base64.URLEncoding.EncodeToString(uuid.NewV3(uuid.Must(uuid.NewV4(), err), buf.String()).Bytes())
The text was updated successfully, but these errors were encountered: