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

アクセス時のHTTPリクエストmethodの厳格化 #279 #324

Merged
merged 21 commits into from
Jun 11, 2021

Conversation

uzulla
Copy link
Collaborator

@uzulla uzulla commented Jun 6, 2021

ref: #279

  • User系コントローラーで各ControllerのHttp method確認を追加
    • Blogs
    • Common
    • Entries
    • (これですべて対応完了と思われる
  • Admin系コントローラーで各ControllerのHttp method確認を追加
    • Blogplugins
    • Blogs
    • BlogSettings
    • Categories
    • Comments
    • Common
    • Files
    • PasswordReset
    • Session
    • SystemUpdate
    • Tags
    • Users
    • (Entriesは事前に対応済み、これですべて対応完了と思われる
  • 明確ではなかった箇所で、明確化したところテストが壊れたので修正
  • 一部、Sigチェック不足や、POSTであるべき所がGETである箇所を発見したのでISSUEやTODO作成。追って直していく。

Note

  • これで各ActionのHttp Methodが固定・確認できたので、安全・安定性と見通しが向上した。
  • 将来的には「アクション実行の手前」で、Http Methodを固定したい
    • 完了すれば、AdhocなSig確認でなく、「POST(更新処理)はすべてSig確認」などを前段で入れるなどができるようになる。
  • ただし、現状は各アクションはGET/POSTが混在する前提で書かれており(参照と、更新がおなじアクションに記述され、引数で分岐している)アクションのリファクタリングと分割を予定
  • GET+Sig有りな更新処理を洗い出したので、これらを対応していく。

作業時間 6h

@uzulla uzulla changed the title Issu279/check http method WIP 各アクセスのHttp methodの厳格化 #279 Jun 6, 2021
@uzulla uzulla changed the title WIP 各アクセスのHttp methodの厳格化 #279 アクセス時のHTTPリクエストmethodの厳格化 #279 Jun 6, 2021
@uzulla uzulla marked this pull request as ready for review June 6, 2021 18:27
@fc2dev fc2dev merged commit 6ee67c1 into fc2blog:master Jun 11, 2021
@uzulla uzulla deleted the issu279/check-http-method branch July 31, 2021 03:51
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