-
-
Notifications
You must be signed in to change notification settings - Fork 211
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 OpenAPI documentation login #1594
Conversation
Of course, there is always the possibility that I'm doing something wrong but: I note also that the image in the top left has disappeared again - wasn't that fixed already? API Debug logs:
|
I did the coding whilst being on the train, it is a very likely possibility that this branch wasn't branched off of the most recent Concerning the other issue: You'll have to have selected the exact same scheme you used to connect to the API at the very top of the screen. It has to match 100% or any modern browser will reject it as cross-site-kind-of-issue. So if you access it as https://pi.hole/api/docs or https://192.168.1.2/api/docs - everything from the scheme to the host needs to be exactly the same. But you may have succeeded in this and the Login button became red only later, this isn't really clear from your post. Then
is the same issue we have had for Putting the PR on hold until I have fixed this and will re-request review then. Thanks! |
Signed-off-by: DL6ER <[email protected]>
…cation to circumvent CORS issues Signed-off-by: DL6ER <[email protected]>
@PromoFaux Should be fixed and also rebased on latest |
Still no joy here:
|
…token in addition) Signed-off-by: DL6ER <[email protected]>
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.
Confirmed latest commit fixes it
What does this implement/fix?
Recently, we switched from the previous challenge-response to password-based authentication (preferred to be sent over HTTPS). In this change, we forgot to adjust the login mechanism in the onboard OpenAPI documentation for Pi-hole's API. This is now done with this PR.
Related issue or feature (if applicable): N/A
Pull request in docs with documentation (if applicable): N/A
By submitting this pull request, I confirm the following:
git rebase
)Checklist:
developmental
branch.