Skip to content
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

Candidate 0.26.1 #6232

Merged
merged 4 commits into from
Jul 17, 2018
Merged

Candidate 0.26.1 #6232

merged 4 commits into from
Jul 17, 2018

Conversation

ohemorange
Copy link
Contributor

No description provided.

bmw and others added 3 commits July 16, 2018 07:54
Fixes #6207.

As noted by Erica:

- we no longer need to check if it exists before linking to it, because we delete properly.
- the previously excisting check on if server is in `LE_REUSE_SERVERS` before unlinking is nice, but probably not necessary, especially since we don't officially support people doing weird things with symlinks in our directories, and because we rmdir which will fail if it's not empty anyway.

* Create single account symlink.

* refactor _delete_accounts_dir_for_server_path

* add symlinked account dir deletion

* add tests

(cherry picked from commit 9b0d271)
@ohemorange ohemorange requested a review from bmw July 17, 2018 22:07
@@ -475,9 +475,11 @@ apache:
Apache Web Server plugin - Beta

--apache-enmod APACHE_ENMOD
Path to the Apache 'a2enmod' binary. (default: None)
Copy link
Member

Choose a reason for hiding this comment

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

More candidates for CERTBOT_DOCS here and below!

Copy link
Member

Choose a reason for hiding this comment

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

See #6234.

@bmw
Copy link
Member

bmw commented Jul 17, 2018

Remember, this PR should ideally be merged and not squashed.

@bmw bmw merged commit 20f8e69 into master Jul 17, 2018
@bmw bmw deleted the candidate-0.26.1 branch July 17, 2018 22:34
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.

None yet

2 participants