search

kenkoooo / AtCoderProblems

Extend your AtCoder
https://kenkoooo.com/atcoder/
MIT License
1.41k stars 150 forks source link

Case insensitive漏れを修正 #1367

Closed hotate29 closed 1 year ago

hotate29 commented 1 year ago

1337 において、SubmissionClientがcase insensitiveになるように変更を行いました。しかし、UsersProblemsTime、UsersAcceptedのリクエストが来た際に発行されるSQLに誤りがあり、不十分なことが分かりました。#1353 の原因になっている可能性があります。

具体的には、

LOWER(user_id) = ANY(SELECT LOWER(u) FROM UNNEST($1) AS a(u))

とするべき部分が、

LOWER(user_id) = ANY($1)

という、誤った状態になっており、対象のユーザー名に大文字が含まれる場合に妙な返答をしてしまいます(https://github.com/kenkoooo/AtCoderProblems/issues/1353#issuecomment-1440089332 )。

変更が不十分だった箇所: https://github.com/kenkoooo/AtCoderProblems/pull/1337/commits/26a01881166f18a518832a1638799c9d03a3cae6#diff-b99c1b312c4e1377b1cb4ab2bafda8553b72d55f19e65a7f5984febd45af1385

kenkoooo commented 1 year ago

デプロイします…

hotate29 commented 1 year ago

ありがとうございます！

kenkoooo commented 1 year ago

デプロイしました！