-
Notifications
You must be signed in to change notification settings - Fork 57
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
Improved documentation around argument default values and basic auth example #107
Comments
[Triage] Adding @prudhvigodithi to get more information on this. |
Hey @Sovietaced thanks for the feedback, can you please open a PR to fix this? |
Yeah sure. Gimme a week. |
Given that this project uses Feels absurd to me that the issue has been laying around for 2 years so I'll just work on adding a basic auth example :( |
Ready for review: #114 |
Thanks @Sovietaced I have assigned this issue to you. |
It was surprisingly challenging for me to get going with basic auth. The default values of the arguments are not configured and it was not clear that I would have to explicitly disabling request signing in order to get basic auth to work (was receiving 403s).
The text was updated successfully, but these errors were encountered: