add ipv6 addresses to dc_relay_nets#78
Open
matthijskooijman wants to merge 1 commit intonamshi:masterfrom
Open
add ipv6 addresses to dc_relay_nets#78matthijskooijman wants to merge 1 commit intonamshi:masterfrom
matthijskooijman wants to merge 1 commit intonamshi:masterfrom
Conversation
Previously, this added all connected ipv4 networks to enable relaying for all direclty connected hosts, but if a host would connect through an ipv6-enabled network and actually uses an ipv6 connection, relay would suddenly be denied. This commit also includes all ipv6 addresses, just like ipv4 addresses. Note that Exim uses : as a separator between entries in this list, so any : inside the ipv6 address must be doubled. This also adds `scope global` to the `ip address show` command, to prevent also picking up link-local addresses. This should not normally affect the ipv4 address, since those are normally global scope anyway.
Author
|
Note that enabling ipv6 on docker is not as trivial as it should be currently. It might be a matter of switching on ipv6 on the default network, but I have a more complicated setup with different networks with static ipv6 configuration, so I can't quickly give you details on how to reproduce this issue. However, I can show you how the |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Previously, this added all connected ipv4 networks to enable relaying for all
direclty connected hosts, but if a host would connect through an ipv6-enabled
network and actually uses an ipv6 connection, relay would suddenly be denied.
This commit also includes all ipv6 addresses, just like ipv4 addresses. Note
that Exim uses : as a separator between entries in this list, so any : inside
the ipv6 address must be doubled.
This also adds
scope globalto theip address showcommand, to prevent alsopicking up link-local addresses. This should not normally affect the ipv4
address, since those are normally global scope anyway.