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

Fix login request, Add headers CertbotStratoApi #36

Merged
merged 2 commits into from
Jun 4, 2024

Conversation

PLanB2008
Copy link

Seems theres an issue when trying to login without a proper User-Agent added to header. I could not login without providing a header. Feel free to merge my request :)

Regards,
PLanB

@Buxdehuda
Copy link
Owner

Is this only relevant for the login request?
I think this should be added to the session object so that all requests use the header by default

@PLanB2008
Copy link
Author

PLanB2008 commented Jun 4, 2024

Agree, you are right. This is way cleaner :)

I added a commit.

@Buxdehuda
Copy link
Owner

Thank you

@Buxdehuda Buxdehuda merged commit dd64080 into Buxdehuda:master Jun 4, 2024
1 check passed
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