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

Allow paths to pi-hole that don't contain /admin #62

Closed
cbundy opened this issue Oct 22, 2022 · 6 comments · Fixed by #68
Closed

Allow paths to pi-hole that don't contain /admin #62

cbundy opened this issue Oct 22, 2022 · 6 comments · Fixed by #68
Assignees

Comments

@cbundy
Copy link
Contributor

cbundy commented Oct 22, 2022

What feature would you like added?

Hello,

My main pi-hole is reachable at something like https://mydomain.com/pihole. Because it's behind a reverse proxy, I ended up hiding the /admin part of the path.

I've built my own version now that has removed it, I can make a PR with a configuration option that allows more control over the URL if you think this would be a welcome addition.

Version

latest

Log output

No response

@mattwebbio
Copy link
Owner

Hi! I would gladly accept a pull request for this. I think the best approach might be to add optional PRIMARY_HOST_PATH and SECONDARY_HOST_(#)_PATH options that default to /admin. How do you feel about this?

The other option could be to replace PRIMARY_HOST_BASE_URL with a singular PRIMARY_HOST_URL that includes the path (i.e. http://10.0.0.2/admin), but this would be a breaking change.

@cbundy
Copy link
Contributor Author

cbundy commented Oct 23, 2022

Yes, agree with your idea of optional path. Probably not too common and avoiding a breaking change is ideal.

@cbundy
Copy link
Contributor Author

cbundy commented Oct 29, 2022

I've created a PR #68 , would love your feedback

@mattwebbio
Copy link
Owner

I've created a PR #68 , would love your feedback

This is awesome! It looks like you've put quite a bit of time into this. I really appreciate it! I'm AFK this weekend but I'll try to look at it some time this coming week 😊

@mattwebbio
Copy link
Owner

Merged and bumped to 1.4! Thanks again for your contribution, @cbundy :)

@cbundy
Copy link
Contributor Author

cbundy commented Nov 7, 2022

Awesome, thanks, pleasure to work with you and happy to do it again sometime

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

Successfully merging a pull request may close this issue.

2 participants