[PR #1095] [MERGED] Add more doc comments for MySQL/PostgreSQL connection URIs #2881

New issue

Closed

opened 2026-03-03 09:08:46 +03:00 by kerem · 0 comments

kerem commented

2026-03-03 09:08:46 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/dani-garcia/vaultwarden/pull/1095
Author: @jjlin
Created: 8/13/2020
Status: ✅ Merged
Merged: 8/13/2020
Merged by: @dani-garcia

Base: master ← Head: db-docs

📝 Commits (1)

385c222 Add more doc comments for MySQL/PostgreSQL connection URIs

📊 Changes

1 file changed (+9 additions, -2 deletions)

View changed files

📝 .env.template (+9 -2)

📄 Description

Note that Diesel implements its own parser for MySQL connection URIs, so it
probably doesn't accept the full range of syntax that would be accepted by
MySQL's client libraries, whereas for PostgreSQL, Diesel simply passes the
connection string/URI to PostgreSQL's libpq for processing.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/dani-garcia/vaultwarden/pull/1095 **Author:** [@jjlin](https://github.com/jjlin) **Created:** 8/13/2020 **Status:** ✅ Merged **Merged:** 8/13/2020 **Merged by:** [@dani-garcia](https://github.com/dani-garcia) **Base:** `master` ← **Head:** `db-docs` --- ### 📝 Commits (1) - [`385c222`](https://github.com/dani-garcia/vaultwarden/commit/385c2227e79e7c84b49037620fc43daf99eb3169) Add more doc comments for MySQL/PostgreSQL connection URIs ### 📊 Changes **1 file changed** (+9 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `.env.template` (+9 -2) </details> ### 📄 Description Note that Diesel implements its own parser for MySQL connection URIs, so it probably doesn't accept the full range of syntax that would be accepted by MySQL's client libraries, whereas for PostgreSQL, Diesel simply passes the connection string/URI to PostgreSQL's libpq for processing. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>