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

docs: forbiddenをレスポンスとして返すのを記載 #114

Merged
merged 1 commit into from
Oct 19, 2023

Conversation

rito528
Copy link
Member

@rito528 rito528 commented Oct 15, 2023

一般ユーザーにアクセスされちゃまずいAPIにforbiddenが返される可能性があることを記載しました。

余談: フォームの一覧とか詳細取得とかに明らかに取得できちゃいけないデータ(例えばwebhookのURL)があるのでそこらへんをどうするかをもう一度考えたほうが良さそう

@rito528 rito528 requested a review from Lucky3028 as a code owner October 15, 2023 14:18
@Lucky3028 Lucky3028 merged commit 40ba42d into main Oct 19, 2023
2 checks passed
@Lucky3028 Lucky3028 deleted the addForbidden branch October 19, 2023 13:06
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