Skip to content

TOTP via PyOTP#11

Open
stefango79 wants to merge 2 commits into
couchtyp:masterfrom
stefango79:totp_token
Open

TOTP via PyOTP#11
stefango79 wants to merge 2 commits into
couchtyp:masterfrom
stefango79:totp_token

Conversation

@stefango79

Copy link
Copy Markdown

Hi,

I did some changes for 2FA. Now it's using PyOTP to generate the access token. I tested it with my Schlundtech account via the Docker version. I also edited the setup.py install_requiremnts, but I didn't test it his way. Is there a better way for Docker than RUN pip install for PyOTP?
This is my first (and probably last) Python program, so I didn't change the parameter name for TOKEN, as I didn't wont to break anything. Maybe the name is not the best anymore.

I tested it a few times with my account and also with more than one domain via the -d option. (domain1.com, *.domain1.com, domain2.com, *.domain2.com).

Let me know, if you need some further testing or some changes.

Best regards from Ruhrpott
Stefan

@stefango79

Copy link
Copy Markdown
Author

I cleaned up the Dockerfile. PIP install was obsolete. Pretty sure it works as expected now.

@dwt

dwt commented Jun 6, 2023

Copy link
Copy Markdown

@couchtyp this looks much more in line with what I would expect from 2fa support. Without something like this, I need to always have an external mechanism that updates the configuration file all the time before invoking certbot, which is quite annoying.

Any reason you don't want to merge this?

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