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

plugin/bind: allow binding particularly link-local addresses (using a scope/zone) #6551

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

riedel
Copy link

@riedel riedel commented Mar 13, 2024

1. Why is this pull request needed and what does it do?

I split off this PR as part of #6547 as it introduces new behaviour. It particularly allows binding link local addresses where a scope/zone is needed (typically the interface name). As it uses ResolveIPAddr hostnames are also passed through. It seems that the downstream bind code will accept this as well

2. Which issues (if any) are related?

This started of as #6536

3. Which documentation changes (if any) need to be made?

I tried to change the documentation

4. Does this introduce a backward incompatible change or deprecation?

This widens the acceptable input. I would doubt that workflows relied on the error behavior, but it is not impossible.

also as a side effect allows using hostnames

Signed-off-by: Till Riedel <riedel@teco.edu>
@riedel riedel changed the title allow binding particularly link-local addresses (using a scope/zone) plugin/bind: allow binding particularly link-local addresses (using a scope/zone) Mar 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant