Skip to content

Update Database doc#89

Merged
windsaenger merged 2 commits into
mainfrom
ionos-deploy-now-support-patch-1
Nov 6, 2025
Merged

Update Database doc#89
windsaenger merged 2 commits into
mainfrom
ionos-deploy-now-support-patch-1

Conversation

@ionos-deploy-now-support

Copy link
Copy Markdown
Contributor

No description provided.

@windsaenger windsaenger left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we can improve this statement: "In some frameworks, to ensure a reliable database connection, (..)":

Either we really make it specific for what frameworks we really recommend this URL_encoding (if we can determine this info for every framework)

Or we can say it this way: "If you have problems with your database connection, then we recommend to always URL-encode the database password.."

@windsaenger windsaenger left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes look good.

@windsaenger windsaenger closed this Nov 6, 2025
@windsaenger windsaenger reopened this Nov 6, 2025
@windsaenger windsaenger merged commit 967481c into main Nov 6, 2025
1 check failed
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