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

just some ideas for fixes / conveniences #79

Open
wants to merge 17 commits into
base: master
Choose a base branch
from

Conversation

cyisfor
Copy link

@cyisfor cyisfor commented Aug 1, 2016

was thinking about using this, but all the manual frittering about with copy and paste was bugging me.

Ideally one would run the command as root, and have it drop permissions before it accesses the network, but the sudo crowd seems to think dropping permissions is eeeevil and it'd be a lot bigger change.

Oh, also -nosudo probably also means -noroot for most people, so I'd have to name it letsencrypt-somethingelse

user added 17 commits August 1, 2016 07:51
… configuration file boilerplate around the prefixes
Doing this with a Makefile won't work good, because it won't delete target files ONLY on error, while leaving them alone if they're intermediate.
I keep telling myself I'll go back to python 2 because at least the shoddy syntax doesn't have a lot of retarded boilerplate for encoding/decoding strings. But I just can't give up that unicode support, even if it is (more) shoddy syntax.
Yeah, maybe I should just go back to python 2. Everything's a bytestring, it's all in utf-8, and urllib is eternally stuck in version 2.
"{0}/directory".format(CA)? really?

Just concatenate...
It should be relatively easy to offer the option to execute the commands, instead of begging the user to do it. I for one generate my private keys with the same account that deals with letsencrypt.org, which technically doesn't protect private keys with unix file permissons, but it's convenient at least. It's nicer anyway, to have a standard syntax for lists of commands anyway, instead of just sticking them in the string and trying to figure out which argument {4} corresponds to.
@djd333
Copy link

djd333 commented Jan 11, 2017

I'm trying to get some information from this thread

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