Skip to content

feat: add punishment endpoints#1

Merged
tksimeji merged 4 commits into
mainfrom
feat/punishments
Jul 21, 2026
Merged

feat: add punishment endpoints#1
tksimeji merged 4 commits into
mainfrom
feat/punishments

Conversation

@acrylic-style

Copy link
Copy Markdown
Member

No description provided.

@acrylic-style
acrylic-style requested a review from tksimeji July 21, 2026 13:53
@tksimeji

Copy link
Copy Markdown
Member

これまでOpenAPIのスキーマ定義のpropertiesは各要素の間に空白行を入れていたから、可読性の高い方に統一したいかも

@tksimeji

Copy link
Copy Markdown
Member

punishmentのtargetNameはプレイヤー名?

@acrylic-style

Copy link
Copy Markdown
Member Author

targetNameはIPアドレスだったりプレイヤー名だったり、処罰の種類によって異なる(SpicyAzisaBanの仕様に準拠してる)

@tksimeji

Copy link
Copy Markdown
Member

actorではなくplayerで表現してそうだから、
20260721000000_add_api_key_actor_and_punishment_scopes.sql
のファイル名だけ変更したらマージしちゃって良さそうね

@tksimeji

Copy link
Copy Markdown
Member

まーじしちゃおう☆
恐らくk8sのシークレット追加しないといけないよね

@acrylic-style

Copy link
Copy Markdown
Member Author

シークレット追加しないといけないね
PUNISHMENTS_DATABASE_URLだけあればいけるはず

@tksimeji

tksimeji commented Jul 21, 2026

Copy link
Copy Markdown
Member

りょうかい〜!MariaDBのユーザーは既に作成済み?
ああ、graphユーザーに権限与えればいいか

@acrylic-style

acrylic-style commented Jul 21, 2026

Copy link
Copy Markdown
Member Author

リードオンリーってわけでもなく、CRUD全部任せてるからspicyazisaban用の認証情報流用していい気がする
graphユーザーに権限与えても〇

@tksimeji

Copy link
Copy Markdown
Member

どのDBの権限が必要?

@acrylic-style

Copy link
Copy Markdown
Member Author

spicyazisaban.*があれば大丈夫なはず

@tksimeji
tksimeji merged commit e6f7012 into main Jul 21, 2026
2 checks passed
@tksimeji
tksimeji deleted the feat/punishments branch July 22, 2026 02:04
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