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

Closed
opened 2025-10-09 18:25:10 +03:00 by OVERLORD · 0 comments
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: masterHead: 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>
OVERLORD added the pull-request label 2025-10-09 18:25:10 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/vaultwarden#3532