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

v3-api.yamlのPasswordのvalidationを修正 #2377

Merged
merged 1 commit into from
Apr 17, 2024

Conversation

NapoliN
Copy link
Contributor

@NapoliN NapoliN commented Apr 13, 2024

PostUserRequestとPostLoginRequestの正規表現が間違っており、パスワード入力で英小文字を受け付けないパターンになっていたのを修正

実装に影響はないが、openapi-generatorでクライアントスクリプトを自動生成したときに引っかかる

@pikachu0310 pikachu0310 enabled auto-merge April 16, 2024 13:53
Copy link
Contributor

@pikachu0310 pikachu0310 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

YAMLファイル内でバックスラッシュを使用する場合は、ダブルバックスラッシュ \ として表現する必要があります。これにより、正規表現としての意味でシングルバックスラッシュとして解釈されます。これにより、ASCIIプリンタブル文字(16進数の20から7E)で、長さが10文字以上32文字以下の文字列にマッチする正規表現が適切に設定されます。

@pikachu0310 pikachu0310 merged commit 3164dea into traPtitech:master Apr 17, 2024
10 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants