レビュー指摘の3件を修正 - #49
Merged
Merged
Conversation
- 時刻同期後に fake-hwclock へ保存し、RTC のない Raspberry Pi で 再起動時に時刻が巻き戻るのを防ぐ - 選択商品のバーコード PDF もキャッシュ対象にし、枠線あり版も 起動時にウォームアップ。商品の追加・更新・削除をイベントで受けて キャッシュを作り直す - Danger Zone のパスワード照合に接続元ごとのレート制限を追加。 連続失敗が上限に達している間は照合せず 429 と Retry-After を返す
Contributor
🎨 Visual Regression Test Results✅ All visual regression tests passed! All page screenshots match the expected baselines. 📊 Test Coverage
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
懐疑的レビューで挙がった指摘のうち、対応すると決めた3件を修正しました。
時刻同期の巻き戻り防止
なお、レビューで「範囲外の時刻が500になる」と指摘した点は誤検知でした。リクエストDTOに Min と Max のバリデーションが既にあり、400が返ります。
バーコードPDFのキャッシュ拡張
Danger Zone パスワード照合のレート制限
検証
補足
Generated by Claude Code