Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: dot not remove symlinked default certificate / private key #1074

Merged
merged 2 commits into from
Dec 15, 2023

Conversation

buchdag
Copy link
Member

@buchdag buchdag commented Dec 15, 2023

If default.crt / default.key were symlinks to certificates managed by this container, they were removed along other symlinks in /etc/nginx/certs. This was an unintentional behaviour.

@buchdag buchdag added the type/fix PR for a bug fix label Dec 15, 2023
@buchdag buchdag self-assigned this Dec 15, 2023
@buchdag
Copy link
Member Author

buchdag commented Dec 15, 2023

@Exagone313

@buchdag buchdag merged commit bc3e672 into main Dec 15, 2023
36 checks passed
@buchdag buchdag deleted the default-certificate branch December 15, 2023 12:46
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
type/fix PR for a bug fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant