pi-hole / FTL

The Pi-hole FTL engine
https://pi-hole.net
Other
1.34k stars 187 forks source link

Fix spurious "resource limit exceeded" messages #1892

Closed DL6ER closed 4 months ago

DL6ER commented 4 months ago

What does this implement/fix?

This PR imports https://thekelleys.org.uk/gitweb/?p=dnsmasq.git;a=commitdiff;h=1ed783b8d7343c42910a61f12a8fc6237eb80417:

Replies from upstream with a REFUSED rcode can result in log messages stating that a resource limit has been exceeded, which is not the case.

and updates the version of the embedded dnsmasq server.


Related issue or feature (if applicable): https://discourse.pi-hole.net/t/pihole-unbound-dnsmasq-validation-failed-resource-limit-exeeded/68388

Pull request in docs with documentation (if applicable): N/A


By submitting this pull request, I confirm the following:

  1. I have read and understood the contributors guide, as well as this entire template. I understand which branch to base my commits and Pull Requests against.
  2. I have commented my proposed changes within the code.
  3. I am willing to help maintain this change if there are issues with it later.
  4. It is compatible with the EUPL 1.2 license
  5. I have squashed any insignificant commits. (git rebase)

Checklist:

pralor-bot commented 4 months ago

This pull request has been mentioned on Pi-hole Userspace. There might be relevant details there:

https://discourse.pi-hole.net/t/pihole-unbound-dnsmasq-validation-failed-resource-limit-exeeded/68388/27

DL6ER commented 4 months ago

Confirmed on Discourse:

I was able to reproduce the error every time and can confirm that it no longer occurs with this version.