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

Don't create IPv6 cache entries when IPV6_ADDRESS is empty #528

Merged
merged 2 commits into from Mar 3, 2019

Conversation

DL6ER
Copy link
Member

@DL6ER DL6ER commented Mar 3, 2019

By submitting this pull request, I confirm the following (please check boxes, eg [X]) Failure to fill the template will close your PR:

Please submit all pull requests against the development branch. Failure to do so will delay or deny your request

  • I have read and understood the contributors guide.
  • I have checked that another pull request for this purpose does not exist.
  • I have considered, and confirmed that this submission will be valuable to others.
  • I accept that this submission may not be used, and the pull request closed at the will of the maintainer.
  • I give this submission freely, and claim no ownership to its content.

How familiar are you with the codebase?:

10


Don't create IPv6 cache entries when IPV6_ADDRESS is empty in setupVars.conf.

Currently, we create empty (::) IPv6 entries when users set BLOCKINGMODE=IP in pihole-FTL.conf when the IPV6_ADDRESS variable is empty (or unset) in setupVars.conf.
This PR ensures that IPv6 entries are not created in IP mode when we don't have a suitable address for them.

This template was created based on the work of udemy-dl.

…rs.conf

Signed-off-by: DL6ER <dl6er@dl6er.de>
@DL6ER DL6ER added this to the v4.3 milestone Mar 3, 2019
@DL6ER DL6ER requested a review from a team March 3, 2019 09:24
@pralor-bot
Copy link

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

https://discourse.pi-hole.net/t/ftl-v4-2-2-4-2-3-consumes-nearly-2x-more-memory-then-v4-11/17775/47

Copy link
Member

@PromoFaux PromoFaux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will approve after spelling change :)

dnsmasq_interface.c Outdated Show resolved Hide resolved
@DL6ER DL6ER requested a review from PromoFaux March 3, 2019 17:07
Co-Authored-By: DL6ER <DL6ER@users.noreply.github.com>
Signed-off-by: DL6ER <dl6er@dl6er.de>
@DL6ER DL6ER force-pushed the fix/IP_blocking_empty_address branch from 92b2aac to af7cf36 Compare March 3, 2019 17:09
@DL6ER DL6ER dismissed PromoFaux’s stale review March 3, 2019 17:09

Applied requested change

@DL6ER DL6ER merged commit b13c77b into development Mar 3, 2019
@DL6ER DL6ER deleted the fix/IP_blocking_empty_address branch March 3, 2019 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants