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

.config/example.ymlに記述可能な"db.extra"のコメントを補強 #15108

Open
1 task
samunohito opened this issue Dec 9, 2024 · 0 comments
Open
1 task
Labels
1️⃣ Good First Issue This is a good problem for your first contribution. 💚Refactor Rewriting code without changing behavior

Comments

@samunohito
Copy link
Member

samunohito commented Dec 9, 2024

Summary

misskey/.config/example.yml

Lines 108 to 127 in 7bdc4e8

# ┌──────────────────────────┐
#───┘ PostgreSQL configuration └────────────────────────────────
db:
host: localhost
port: 5432
# Database name
db: misskey
# Auth
user: example-misskey-user
pass: example-misskey-pass
# Whether disable Caching queries
#disableCache: true
# Extra Connection options
#extra:
# ssl: true

上記に記述されているextraについてのコメントを増強する必要があると考えています。
このextraはMisskeyのconfig -> TypeORM(DataSourceクラス) -> pgの流れでリレーされるもので、pgが作るコネクションプールのオプションとして使用されます。このオプションは、既に記載されているsslのほかにもコネクションタイムアウト秒数やコネクションプール最大数など有益なものを含んでいます。

なので…コネクションプール最大数とタイムアウト秒数の設定サンプルと、pgの該当ドキュメントへのリンクをコメントとして追記しておくと親切かと考えました。

Purpose

情報が充実してdb.extraをより有効に活用できるサーバ管理者が増える

Do you want to implement this feature yourself?

  • Yes, I will implement this by myself and send a pull request
@samunohito samunohito added 1️⃣ Good First Issue This is a good problem for your first contribution. 💚Refactor Rewriting code without changing behavior labels Dec 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1️⃣ Good First Issue This is a good problem for your first contribution. 💚Refactor Rewriting code without changing behavior
Projects
Development

No branches or pull requests

1 participant