mirror of
https://github.com/dani-garcia/vaultwarden.git
synced 2026-04-26 01:35:54 +03:00
[PR #1789] [CLOSED] Only create private key in pem format if it doesn't already exist #3005
Labels
No labels
SSO
Third party
better for forum
bug
bug
documentation
duplicate
enhancement
future Vault
future Vault
future Vault
good first issue
help wanted
low priority
notes
pull-request
question
troubleshooting
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/vaultwarden#3005
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/dani-garcia/vaultwarden/pull/1789
Author: @Bobonium
Created: 6/22/2021
Status: ❌ Closed
Base:
main← Head:main📝 Commits (1)
d5cde07Only create private key in pem format if it doesn't already exist📊 Changes
1 file changed (+13 additions, -11 deletions)
View changed files
📝
src/main.rs(+13 -11)📄 Description
The original logic checks if the private and public key are available in the
.derformat, but ignores the private key in.pemformat.This change does not have any impact on the already working setups, it merely enables the User to only provide a single private key in
.pemformat.My use case is simple. I deploy vaultwarden on kubernetes through terraform and I'm unable to create private keys in
.derformat directly. By simply checking for an existing.pemkey first, I can simply mount the key that I've created with terraform and on startup the application will automatically convert it to the correct format.Since the
CONFIGprovides methods to get the key names I also switched to using those instead of generating the names.🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.