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 PiHole V6 /api subdomain #751

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

that1matt
Copy link

linuxserver.io


  • I have read the contributing guideline and understand that I have made the correct modifications

Description

Updating to PiHole v6 using a subdomain, caused 500 errors when trying to load the queries page, or make API calls.

Benefits of this PR and context

This change allows for PiHole v6+ to continue to work behind a subdomain reverse proxy.

How Has This Been Tested?

With the current setup, when accessing /admin/queries or the Query Log tab on the right, you would get a pop up showing an error.
Screenshot 2025-02-18 at 3 53 04 PM
With this new /api block, the Query Log loads as expected.

Source / References

PiHole v6 Release Notes

@cpaucar3
Copy link

cpaucar3 commented Mar 2, 2025

Confirmed that this solution solved the same issue I was facing after upgrading Pi-hole. Thanks!

@gabriel-lando
Copy link
Contributor

Worked for me :) I was not able to login on pihole via swag/nginx since I upgraded to v6. Uncommenting the lines on this PR fixed my issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

3 participants