gridscaleinc / mitty-server

0 stars 0 forks source link

Panic -- getCurrentUserId #63

Closed domanthan closed 7 years ago

domanthan commented 7 years ago

event.go の EventFetchingHandlerにfilters.GetCurrentUserIDを呼び出したら、panicになった。 認証済みhttpRequestしか呼び出せないか?

mitty-server | 2017/07/27 16:25:39 stderr: 2017/07/27 16:25:39 http: panic serving 172.17.0.1:54148: interface conversion: interface is nil, not int mitty-server | 2017/07/27 16:25:39 stderr: goroutine 23 [running]: mitty-server | 2017/07/27 16:25:39 stderr: net/http.(*conn).serve.func1(0xc420056380) mitty-server | 2017/07/27 16:25:39 stderr: /usr/local/go/src/net/http/server.go:1491 +0x12a mitty-server | 2017/07/27 16:25:39 stderr: panic(0xac7080, 0xc4202f6300) mitty-server | 2017/07/27 16:25:39 stderr: /usr/local/go/src/runtime/panic.go:458 +0x243 mitty-server | 2017/07/27 16:25:39 stderr: mitty.co/mitty-server/app/filters.GetCurrentUserID(0xc4200b31d0, 0x3) mitty-server | 2017/07/27 16:25:39 stderr: /go/src/mitty.co/mitty-server/app/filters/auth.go:57 +0xde mitty-server | 2017/07/27 16:25:39 stderr: mitty.co/mitty-server/app/controllers.EventFetchingHandler(0xf56a20, 0xc420402f80, 0xc4200b31d0)

domanthan commented 7 years ago

認証済みであれば、userIdを取得し、認証済みじゃなかったら0か、エラーを返すようにしてお欲しい。

dongri commented 7 years ago

修正しました。認証エラーではなく、Requiredチェックのエラーを返すところでバグってました。直して反映しました。

dongri commented 7 years ago
curl -X POST -H 'X-Mitty-AccessToken: NckLq0NQKkGJ6ScT6ufFdqhYZYnYbh8MYCgCo6CX' http://dev.mitty.co/api/new/event

これで確認できます。

domanthan commented 7 years ago

解消