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

chapter2 section1 の修正 #93

Open
Takeno-hito opened this issue Jul 18, 2023 · 4 comments
Open

chapter2 section1 の修正 #93

Takeno-hito opened this issue Jul 18, 2023 · 4 comments

Comments

@Takeno-hito
Copy link
Member

  • 1-3 err := にするべきところ err = になっている箇所がある
  • source .env をやめたい
  • salt papper(secret salt) を混同していたので修正する
  • handler.go ではなく handler package にする
  • whoami を me にする
  • 使用しない import がある
  • e.Group("") が期待した通りの動きをしてくれなさそうなので調査修正
  • Parsetime true をいれるかハッシュ化のライブラリを変えるかする
  • signUpHandler loginHandler のケース修正
@Takeno-hito
Copy link
Member Author

#90 でやってもらってたわ ごめん

@hijiki51
Copy link
Member

このコードのサイズでパッケージ切る理由はあんまりない気がする(切ってもいいけど)

@pikachu0310
Copy link
Contributor

e.Group("") が期待した通りの動きをしてくれなさそうなので調査修正
についてもう少し詳しく知りたいです @Takeno-hito

@Takeno-hito
Copy link
Member Author

Takeno-hito commented Jul 26, 2023

このコードのサイズでパッケージ切る理由はあんまりない気がする(切ってもいいけど)

パッケージの切り方も学んでほしいし、1ファイルにするにしては少し大きいなと思って提案してます

e.Group("") が期待した通りの動きをしてくれなさそうなので調査修正
についてもう少し詳しく知りたいです @Takeno-hito

e.Group("") を使うと、グループの外のハンドラーにも middleware が適用されるっぽい?(不具合っぽそうだけど)
そもそもこれは同じプレフィックスのをまとめたいだけだから middleware の適用のために e.Group("") を使うのが変だと思うけど、ちょっと自信がないので要調査、という感じです

ref( traP only): https://q.trap.jp/messages/2e48f7ad-118b-4242-bf28-b555b63775f5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants