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

client: getting a non-existing client raises 500 instead of 404 #348

Closed
waynerobinson opened this issue Jan 3, 2017 · 0 comments
Closed
Labels
bug Something is not working.
Milestone

Comments

@waynerobinson
Copy link

21:26:20 hydra.1    | time="2017-01-03T21:26:20+10:00" level=info msg="started handling request" method=GET remote="[::1]:63928" request="/clients/partner_ui_28"
21:26:20 hydra.1    | time="2017-01-03T21:26:20+10:00" level=error msg="An error occurred" error=": Not found" request_id=d3a57692-1ed6-40f3-aee7-8cc361838b59 status=500
21:26:20 hydra.1    | time="2017-01-03T21:26:20+10:00" level=debug msg="Stack trace: \ngithub.jparrowsec.cn/ory-am/hydra/client.(*SQLManager).GetConcreteClient\n\t/Users/waynerobinson/.go/src/github.com/ory-am/hydra/client/manager_sql.go:121\ngithub.jparrowsec.cn/ory-am/hydra/client.(*Handler).Get\n\t/Users/waynerobinson/.go/src/github.com/ory-am/hydra/client/handler.go:149\ngithub.jparrowsec.cn/ory-am/hydra/client.(*Handler).Get-fm\n\t/Users/waynerobinson/.go/src/github.com/ory-am/hydra/client/handler.go:35\ngithub.jparrowsec.cn/ory-am/hydra/vendor/github.com/julienschmidt/httprouter.(*Router).ServeHTTP\n\t/Users/waynerobinson/.go/src/github.com/ory-am/hydra/vendor/github.com/julienschmidt/httprouter/router.go:299\ngithub.jparrowsec.cn/ory-am/hydra/vendor/github.com/urfave/negroni.Wrap.func1\n\t/Users/waynerobinson/.go/src/github.com/ory-am/hydra/vendor/github.com/urfave/negroni/negroni.go:41\ngithub.jparrowsec.cn/ory-am/hydra/vendor/github.com/urfave/negroni.HandlerFunc.ServeHTTP\n\t/Users/waynerobinson/.go/src/github.com/ory-am/hydra/vendor/github.com/urfave/negroni/negroni.go:24\ngithub.jparrowsec.cn/ory-am/hydra/vendor/github.com/urfave/negroni.middleware.ServeHTTP\n\t/Users/waynerobinson/.go/src/github.com/ory-am/hydra/vendor/github.com/urfave/negroni/negroni.go:33\ngithub.jparrowsec.cn/ory-am/hydra/vendor/github.com/urfave/negroni.(middleware).ServeHTTP-fm\n\t/Users/waynerobinson/.go/src/github.com/ory-am/hydra/vendor/github.com/urfave/negroni/negroni.go:33\nnet/http.HandlerFunc.ServeHTTP\n\t/usr/local/Cellar/go/1.6.2/libexec/src/net/http/server.go:1618\ngithub.jparrowsec.cn/ory-am/hydra/cmd/server.(*Handler).rejectInsecureRequests\n\t/Users/waynerobinson/.go/src/github.com/ory-am/hydra/cmd/server/handler.go:133\ngithub.jparrowsec.cn/ory-am/hydra/cmd/server.(*Handler).(github.com/ory-am/hydra/cmd/server.rejectInsecureRequests)-fm\n\t/Users/waynerobinson/.go/src/github.com/ory-am/hydra/cmd/server/handler.go:54\ngithub.jparrowsec.cn/ory-am/hydra/vendor/github.com/urfave/negroni.HandlerFunc.ServeHTTP\n\t/Users/waynerobinson/.go/src/github.com/ory-am/hydra/vendor/github.com/urfave/negroni/negroni.go:24\ngithub.jparrowsec.cn/ory-am/hydra/vendor/github.com/urfave/negroni.middleware.ServeHTTP\n\t/Users/waynerobinson/.go/src/github.com/ory-am/hydra/vendor/github.com/urfave/negroni/negroni.go:33\ngithub.jparrowsec.cn/ory-am/hydra/vendor/github.com/urfave/negroni.(middleware).ServeHTTP-fm\n\t/Users/waynerobinson/.go/src/github.com/ory-am/hydra/vendor/github.com/urfave/negroni/negroni.go:33\ngithub.jparrowsec.cn/ory-am/hydra/vendor/github.com/meatballhat/negroni-logrus.(*Middleware).ServeHTTP\n\t/Users/waynerobinson/.go/src/github.com/ory-am/hydra/vendor/github.com/meatballhat/negroni-logrus/middleware.go:135\ngithub.jparrowsec.cn/ory-am/hydra/vendor/github.com/urfave/negroni.middleware.ServeHTTP\n\t/Users/waynerobinson/.go/src/github.com/ory-am/hydra/vendor/github.com/urfave/negroni/negroni.go:33\ngithub.jparrowsec.cn/ory-am/hydra/vendor/github.com/urfave/negroni.(*Negroni).ServeHTTP\n\t/Users/waynerobinson/.go/src/github.com/ory-am/hydra/vendor/github.com/urfave/negroni/negroni.go:73\nnet/http.serverHandler.ServeHTTP\n\t/usr/local/Cellar/go/1.6.2/libexec/src/net/http/server.go:2081\nnet/http.(*conn).serve\n\t/usr/local/Cellar/go/1.6.2/libexec/src/net/http/server.go:1472\nruntime.goexit\n\t/usr/local/Cellar/go/1.6.2/libexec/src/runtime/asm_amd64.s:1998"
@aeneasr aeneasr added the bug Something is not working. label Jan 3, 2017
@aeneasr aeneasr added this to the 1.0.0: stable release milestone Jan 3, 2017
@aeneasr aeneasr changed the title Getting a non-existing client raises 500 instead of 404 client: getting a non-existing client raises 500 instead of 404 Jan 3, 2017
aeneasr pushed a commit that referenced this issue Jun 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is not working.
Projects
None yet
Development

No branches or pull requests

2 participants