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

Update jwt-go depdenceny #51

Closed
aeneasr opened this issue Jun 18, 2016 · 9 comments
Closed

Update jwt-go depdenceny #51

aeneasr opened this issue Jun 18, 2016 · 9 comments

Comments

@aeneasr
Copy link
Member

aeneasr commented Jun 18, 2016

src/github.com/ory-am/fosite/token/jwt/jwt.go:29: cannot use claims.ToMap() (type map[string]interface {}) as type jwt.Claims in assignment:
map[string]interface {} does not implement jwt.Claims (missing Valid method)
@plandem
Copy link

plandem commented Jun 18, 2016

same issue on go 1.6.1

@aeneasr
Copy link
Member Author

aeneasr commented Jun 18, 2016

thank you for the report @plandem . Please use the 👍 feature if this issue affects you in the future. The issue is not related to the go version but due to breaking changes in jwt-go. You can use glide install to install the right dependencies.

@plandem
Copy link

plandem commented Jun 18, 2016

sorry, i'm still quite new to Go. i choose https://github.com/kovetskiy/manul as dep manager, but anyway, i still can't understand how to fix errors.

@aeneasr
Copy link
Member Author

aeneasr commented Jun 18, 2016

No worries, this repository uses glide for dependency management: https://github.com/Masterminds/glide

@plandem
Copy link

plandem commented Jun 18, 2016

i still can't understand how to fix issue. i'm trying to compile example and just no luck. tried glide too

@aeneasr
Copy link
Member Author

aeneasr commented Jun 18, 2016

Thanks for the report, there indeed seems to be something broken with running the example. Additionally, the docs are missing glide. Sorry for that.

Patch #52 resolves the internal import in the example. Now do the follwoing to run the example with glide:

go get github.com/Masterminds/glide
go get github.com/ory-am/fosite
# unix
cd $GOPATH/src/github.com/ory-am/fosite
# windows
cd %GOPATH%\src\github.com\ory-am\fosite
glide install
go run fosite-example/main.go

Thank you for your patience and reports :) I hope everything works now. If not, let me know.

@plandem
Copy link

plandem commented Jun 18, 2016

worked finally. thanks.

p.s.: it would be nice to see some hints for 2 step auth process :)

@aeneasr
Copy link
Member Author

aeneasr commented Jun 18, 2016

yeah aboslutely. the one thing this repository is missing is good docs. it will be done, promised :)

@aeneasr
Copy link
Member Author

aeneasr commented Jun 20, 2016

resolved in #53

@aeneasr aeneasr closed this as completed Jun 20, 2016
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