Feedback for "Allow defining clients by their MAC address, host name and networking interface"

Yep, I think I found where the issue was, and have prepared a fix for this here:

1 Like

Is it still running? We're currently haunting another issue with incorrect database pointers when a massive number of TCP clients is spawned. The new/mac_client branch may not be fully up-to-date with respect to development so this may have already been fixed.

Yes, still running since 2 days now. Maybe it was an unrelated glitch. It seems my issues are resolved for now. I will continue using new/mac_client and report new issues.

For now, thank you for your help and excellent work!

1 Like

As version 5.1.1 is out and there are some bugs already fixed in development branch, i wanted to ask you to update the branch with latest commits?
Especially updating of DNS settings and conditional forwarding is broken at the moment within this branch.

Thanks for your hard work! I like this project very much and as dual-stack user this branch is very useful (happy wife, happy life :smile: )


I second this request.


Thanks for the feedback and hard work, we will wait for it :smile:

Currently I am on:

pi@pihole:~ $ pihole -v
  Pi-hole version is development v5.0-134-g60f0ab51 (Latest: v5.1.1)
  AdminLTE version is new/mac_clients v5.0-403-ga901489a (Latest: v5.1)
  FTL version is new/mac_clients vDev-661dad9 (Latest: v5.1)

Am I understanding that now the best way to keep up with this version to switch over to the dev branch? Is that by doing:

pihole checkout dev

or do I need to do each component:

pihole checkout core dev
pihole checkout web dev
pihole checkout ftl dev

Or should I stick on this branch for now? I know there was DB changes with this, so don't want to corrupt things by switching out of this branch prematurely.

It has not been merged now. I'm sure, DL6ER will post a message here when he merged the branches into development.

For now you should stay on this branch if you don't want to loose the ability to define clients by MAC, hostname, etc. and want to avoid to delete your long-term-database.

This two will be sufficient.

Current status is: Waiting for FTL#845 to get merged.

Then I will update all the features branches floating around. And I will also mark them as ready-to-be-merged into development.

Depending on the team's review abilities this may happen within minutes to hours to days. I hope that we can release FTL v5.1.1 tomorrow which also means you will get all the updated branches tomorrow.

I could already do it now, however, there are two drawbacks:

  1. I have double amount of work (I don't care too much about this, but see no. 2)
  2. There would be a needless extra merge commit. It is just cleaner to wait for release/v5.1.1 to be done in order to reduce the number of merge commits floating around. They are just noise. However,. given the complexity of the currently discussed branches, a rebase would result in even (much) more work than an ordinary merge which is why I'll prefer the latter.

I hope this makes my motivation clearer. It's not about me simply not wanting to do it.

Sorry if my post came across in any way other than just seeking clarification. I totally understand that in the world of software development, things take time. I also get that cleaner is better, and would wish extra work on no one.

I do want to take a moment to thank you (and the rest of the team) on putting this all together. Specifically this feature is a really great one, that is helping to fine tune my setup for the different use cases in my household.


I don't think your post comes across as demanding in any way. Re-reading his reply, I don't think he received it this way, either, so all good.

There seems to be a lot of work going on behind the scenes. They added another optimization yesterday and still have one more bugfix prepared and even an update for the dnsmasq core to the (today released!!) dnsmasq v2.82.

How cool it this!

Also, @DanSchaper commented that the update of dnsmasq will push FTL to v5.2 so v5.1.1 will maybe be skipped. We can stay tuned for more awesome Pi-hole things to come.

This feature branch (new/mac_clients) has been updated to the latest version of FTL (release/v5.2).

@DL6ER Thank you very much! Will the AdminLTE also be updated so changes (for example Conditional Forwarding on DNS page) can be updated?

Thanks for your hard work :slight_smile:

Yes, however, the AdminLTE branch is still awaiting two more PRs to be reviewed + merged, first.

Alright, thank you very much for your fast reply :slight_smile:

PRs have been merged, so I just updated the web branch as well now. Please test if adding custom IPs still works, I had to resolve some merge conflicts and I did so with a "best efforts" approach. I'm limited to my mobile development system right now so I cannot really do much to test my changes. If adding custom IPs in the dropdown select menu still works (Groups -> Clients page), then everything went well.

Just tried it, seems to work! Thank you very much :slight_smile:

Thank you very much for pushing the new release to this. pihole is working exceedingly well now. From my experience testing release/5.2, it seems to fix alot of stuff.

One further addition is to ask the kernel immediately (through Netlink) if it knows the MAC address of a new client before doing the group lookups in the database. This should make recognizing clients by their MAC addresses finally pretty reliable.

Please test this some more, but I have the feeling that this feature has become ready for inclusion in the v5.3 release.