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

Update logging and authentification #9

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

Update logging and authentification #9

wants to merge 2 commits into from

Conversation

SapuSeven
Copy link
Contributor

Updated logging to display more information about the executed requests and changed the authentication method.

@AntonioCS
Copy link
Owner

Hey,
Sorry for not doing anything regarding this pull request.
I haven't used this script in quite some time. So is there a new way to authenticate?

@AntonioCS
Copy link
Owner

I just saw your issue. Have you tested this in another environment that is not Raspberry pi?

@SapuSeven
Copy link
Contributor Author

I am so sorry, I completely missed your reply.

I just tested it some more and the old authentication works fine on Manjaro Linux.

However, I think that providing the credentials by using wget parameters is a better way as it seems like in some cases wget interprets the hostname after the @ as a port, which leads to my issue. Using parameters is definitely a way to improve overall compatibility.

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