chapter2 section1 の修正
Takeno-hito opened this issue · comments
- 1-3 err := にするべきところ err = になっている箇所がある
- source .env をやめたい
- salt papper(secret salt) を混同していたので修正する
- handler.go ではなく handler package にする
- whoami を me にする
- 使用しない import がある
- e.Group("") が期待した通りの動きをしてくれなさそうなので調査修正
- Parsetime true をいれるかハッシュ化のライブラリを変えるかする
- signUpHandler loginHandler のケース修正
#90 でやってもらってたわ ごめん
このコードのサイズでパッケージ切る理由はあんまりない気がする(切ってもいいけど)
e.Group("") が期待した通りの動きをしてくれなさそうなので調査修正
についてもう少し詳しく知りたいです @Takeno-hito
このコードのサイズでパッケージ切る理由はあんまりない気がする(切ってもいいけど)
パッケージの切り方も学んでほしいし、1ファイルにするにしては少し大きいなと思って提案してます
e.Group("") が期待した通りの動きをしてくれなさそうなので調査修正
についてもう少し詳しく知りたいです @Takeno-hito
e.Group("") を使うと、グループの外のハンドラーにも middleware が適用されるっぽい?(不具合っぽそうだけど)
そもそもこれは同じプレフィックスのをまとめたいだけだから middleware の適用のために e.Group("")
を使うのが変だと思うけど、ちょっと自信がないので要調査、という感じです
ref( traP only): https://q.trap.jp/messages/2e48f7ad-118b-4242-bf28-b555b63775f5