isucon / isucon11-qualify

ISUCON11 予選 (ISUCONDITION)
https://isucon.net/archives/55821036.html
MIT License
166 stars 83 forks source link

fix(bench): postの指定をされた場合ははじく #1355

Closed Nagarei closed 3 years ago

Nagarei commented 3 years ago

やったこと

対応issue

close #1326

セルフチェック

備考

buchy commented 3 years ago

@Nagarei ci死んでそうなので、動くように別PRで試してみました。問題なさそうであればこちらに取り込んでください。 https://github.com/isucon/isucon11-qualify/pull/1358

改修内容自体は問題なさそうです

buchy commented 3 years ago

別途PR出したやつちゃんと動いてなかったのでわすれてください。 本番ベンチはuseTLSオプションついてるからその時だけport厳しくするのとかありかも?

Nagarei commented 3 years ago

TLSのときだけ厳しくする方向で修正しました