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
OAuth Browsable API Login #188
base: master
Are you sure you want to change the base?
Conversation
@@ -60,15 +60,16 @@ MANAGER_IP="$(hostname -I | cut -d' ' -f1)" | |||
# Set to OAUTH if using OAuth Password Flow Authentication, callback url needs to be api/v2/results | |||
CALLBACK_AUTHENTICATION=TOKEN | |||
|
|||
CLIENT_ID=sensor01.sms.internal | |||
CLIENT_ID=sensor01.sms.internal # must match FQDN |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you not just use socket.getfqdn(). Will that not work in Docker?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not able to get the right value in docker
Note the following still need to be implemented:
|
Adds ability to log in to sensor browsable API using OAuth. The user logs in using OAuth 2 Authorization Code flow. Then the token is stored in the session.