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

Pi-hole v6 Project #1358

Closed
PromoFaux opened this issue Jul 4, 2023 · 3 comments
Closed

Pi-hole v6 Project #1358

PromoFaux opened this issue Jul 4, 2023 · 3 comments
Labels

Comments

@PromoFaux
Copy link
Member

PromoFaux commented Jul 4, 2023

Project can be found here: https://github.com/orgs/pi-hole/projects/22/views/1

@PromoFaux PromoFaux changed the title Test Docker Pi-hole V6 Tracker Jul 4, 2023
@PromoFaux PromoFaux pinned this issue Jul 4, 2023
@PromoFaux PromoFaux added the v6 label Jul 4, 2023
@PromoFaux
Copy link
Member Author

@pi-hole/docker-maintainers work towards V6 will be happening in development-v6.

Looking at this as a good opportunity to start from the ground up. You may or may not have been following along with v6-alpine-play - it was mostly just an experiment to see "What if?" (And there have been calls to go alpine-based for years), now is as good a time as any.

There is still a long way to go, and lots of things that need doing (Tests, anyone? I am bad at tests...) which I will try to keep updated above, but for now the image builds and runs and appears to work... That is to say, It's at a stop where I'm going to stop directly pushing to the branch, which should hopefully go some way to making it easier to keep up with development via PRs, and even contribute... if anyone feels like dusting off their ol' coding fingers.

@edgd1er
Copy link
Contributor

edgd1er commented Jul 11, 2023

I'm just starting to play with this version, I was wandering if variables, as it is in the current version, will be uppercase.
For the moment I added a simple lowercase transformation in the start script (tr):

  declare -px | grep FTLCONF_ | sed -E 's/declare -x FTLCONF_([^=]+)=\"(|.+)\"/\1 \2/' | tr '[:upper:]' '[:lower:]' | while read -r name value

I'm afraid with variables using upper and lower cases, something more efficient will be required. Here are variables with both lower and upper case, many other exist:

dns.EDNS0ECS = true
dns.ignoreLocalhost = false
dns.showDNSSEC = true
dns.analyzeOnlyAandAAAA = false

May be you already have an idea for processing FTL_ ?

@PromoFaux
Copy link
Member Author

I was wandering if variables, as it is in the current version, will be uppercase.

I guess they don't have to be. So far in my testing I have just been putting them in the same casing as they are in the FTL API docs but prefixed with FTLCONF_ as we have in previous verisons. e.g: FTLCONF_revserver_active: 'true' (replacing the . in the config for _ in the environment variable

@PromoFaux PromoFaux unpinned this issue Jul 19, 2023
@PromoFaux PromoFaux pinned this issue Jul 19, 2023
@PromoFaux PromoFaux changed the title Docker Pi-hole V6 Tracker Pi-hole v6 Project Jul 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants