-
Notifications
You must be signed in to change notification settings - Fork 249
docs(tutorials): fix errors in the tutorial content vaultwarden-serverless #5374
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
base: main
Are you sure you want to change the base?
Conversation
Co-authored-by: Jessica <[email protected]>
Co-authored-by: Rowena Jones <[email protected]>
| | CPU | 100 mVCPU | | ||
| | RAM | 128 MB | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove CPU/RAM requirements as it depends on actual usage, lowest values may harm performance in several scenarios
|
|
||
| 4. Set the following ¨¨**secrets**: | ||
| | Key | Value | | ||
| |----------------|-----------------------------------------------------------------------------------------------------------------------------------| | ||
| | `DATABASE_URL` | connection string retrieved during the [previous step](#creating-a-serverless-sql-database) | | ||
| | `DATABASE_URL` | connection string retrieved during the [previous step](#creating-a-serverless-sql-database) (IPv4 only) | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove this change as the tutorial relies on a Serverless Database and there is no notion of IP involved
| | `USER_ATTACHMENT_LIMIT` | `0` | | ||
| | `SENDS_ALLOWED` | `false`| |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Personal configuration, should be removed from this table, and can be added as a note instead
Description
Added port and database requirements