Repository navigation
docs: Wiki をリポジトリ内 docs で管理する - #402
Merged
Merged
Conversation
kosumo77
reviewed
Oct 28, 2026
Comment on lines
+9
to
+10
| - URL の prefix として `https://id.maximum.vc/oauth/resources` をつけてください | ||
| - 例えば、 `GET /authuser` なら `https://id.maximum.vc/oauth/resources/authuser` に GET を投げてください |
There was a problem hiding this comment.
https://id.maximum.vc/oauth/resourcesの部分がhttps://api.id.maximum.vc/oauth/resourcesになると思います
Suggested change
| - URL の prefix として `https://id.maximum.vc/oauth/resources` をつけてください | |
| - 例えば、 `GET /authuser` なら `https://id.maximum.vc/oauth/resources/authuser` に GET を投げてください | |
| - URL の prefix として `https://api.id.maximum.vc/oauth/resources` をつけてください | |
| - 例えば、 `GET /authuser` なら `https://api.id.maximum.vc/oauth/resources/authuser` に GET を投げてください |
| これらのパスは開発時にのみ利用可能です。 | ||
| Preview, Production 環境では利用できません。 | ||
|
|
||
| ## `/dev/cron` |
Member
Author
There was a problem hiding this comment.
そもそも Cron Trigger の query parameter で cron=0+*+*+*+* みたいに実行時間指定する方式やめて実行処理名叩く方式になったので、もはや便利機能じゃないから docs から消しちゃっていいな
| iat: number, // 発行日時 | ||
| at_hash: string, // Access Token のハッシュ | ||
| nonce?: string, // Authorization Endpoint に渡された nonce | ||
| authTime?: number, // ユーザーの最終認証日時 |
There was a problem hiding this comment.
signJWTに含むpayloadに使っているのはauth_timeなのでこの部分authTimeではなくてauth_timeだと思います
Suggested change
| authTime?: number, // ユーザーの最終認証日時 | |
| auth_time?: number, // ユーザーの最終認証日時 |
|
|
||
| // email scope が存在している場合に含まれる情報 | ||
| email?: string, // 連絡の取れるメールアドレス | ||
| email_verified: boolean, // false (現状固定) |
There was a problem hiding this comment.
#382 からread:rolesが追加されたので追加してもいいんじゃないかなぁと思います
Suggested change
| email_verified: boolean, // false (現状固定) | |
| email_verified: boolean, // false (現状固定) | |
| // read:roles scope が存在している場合に含まれる情報 | |
| roles?: string[], // ユーザーに割り当てられている Role slug (admin, member, accountant, ...) |
Member
Author
There was a problem hiding this comment.
resource-endpoints.md があるのでこっち参照してねでいい気がするな
Comment on lines
+151
to
+154
| #### invalid_request: Authorization header is not allowed | ||
|
|
||
| `Authorization: foo` がリクエストヘッダに付与されています。 | ||
| 外してください。 |
There was a problem hiding this comment.
#319 で Authorization header is not allowedがMissing client_id or client_secretに代わっているのでそれに合わせる形で更新が必要そうです。
またInvalid Authorization Header が追加されているのでこちらもです。
Suggested change
| #### invalid_request: Authorization header is not allowed | |
| `Authorization: foo` がリクエストヘッダに付与されています。 | |
| 外してください。 | |
| #### invalid_request: Missing client_id or client_secret | |
| POST body に `client_id`, `client_secret` が含まれておらず、リクエストヘッダにも `Authorization: Basic <token>` が付与されていないようです。 | |
| どちらかの方法で指定してください。 | |
| #### invalid_request: Invalid Authorization Header | |
| リクエストヘッダの `Authorization: Basic <token>` の形式がおかしいみたいです。 | |
| `<client_id>:<client_secret>` を Base64 エンコードしたものを指定しているか確認してください。 |
Member
Author
|
よく見てらっしゃる、えらい |
Deploying id with
|
| Latest commit: |
aafbba6
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://297685b6.id-131.pages.dev |
| Branch Preview URL: | https://docs-wiki-to-docs.id-131.pages.dev |
kosumo77
approved these changes
Oct 29, 2026
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.
Wiki は編集が楽だが IdP のバージョンごとに管理できないのがちょっと辛いかも
マージ後 Wiki は消す予定です
長い README.md もいくつかに分割して docs 配下に置いてもよいかもしれない
ところで biome で markdown 対応したんですね