-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
snow: Fix token reference in basic authentication #59315
Conversation
@n3pjk this PR contains the following merge commits: Please rebase your branch to remove these commits. |
@n3pjk this PR contains the following merge commits: Please rebase your branch to remove these commits. |
@n3pjk I restarted the CI and now it is PASSED for this change. Let me know once you are ready for the change. Thanks. |
* Fix OAUTHClient logic * Add Env variable support for snow modules Fixes: ansible#59299 Signed-off-by: Paul Knight <paul.knight@state.de.us> Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
@Akasurde If this patch fixes basic auth, there should be no errors referencing tokens when using basic auth, then this is ready to go. I have retested and OAuth is working correctly. We just need basic retested. |
@n3pjk Basic Auth works for me. Let me know once you are ready to merge this. Thanks. |
In that case, it's ready to merge! |
SUMMARY
Basic authentication does not use tokens, only OAuth. ServiceNowClient constructor code had indent issue which attempted to initialize token regardless of authentication method.
Fixes #59299
ISSUE TYPE
COMPONENT NAME
service_now.py
ADDITIONAL INFORMATION
See #59299 for repro instructions.