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

Traffic ipv6 translation support #7981

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Ashus
Copy link

@Ashus Ashus commented Oct 18, 2024

This adds a support for resolving IPv6 reverse addresses.
Second commit is a bit more controversial, but handy as well. Imagine you have static IPv6 prefixes delegated in your network. For that you need static IPv6 routes to properly configured home routers. This feature allows to translate a dynamically created IP via SLAAC into last part of IPv6 address@GatewayName. eg. fc00:106:4:0:fc82:c2ff:ffff:a0cb would be translated to a0cb@ClientsName.

@AdSchellevis
Copy link
Member

I don't mind taking a look at de69071, but the other commit looks to convoluted to me. can you open a PR for the reverse dns part only?

@Ashus
Copy link
Author

Ashus commented Oct 18, 2024

Here you go. Although I hope you give the second commit a second look later.

@@ -0,0 +1,21 @@
#!/usr/local/bin/php
Copy link
Member

Choose a reason for hiding this comment

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

we would rather fold this special case return into an existing script. duplicating functionality for a single IP family is not ideal and we can easily build a plugin-based wrapper that can be called with pluginctl.

Copy link
Author

Choose a reason for hiding this comment

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

Please point me in the right direction, where you want it, I'm not familiar with this.
I was trying to fetch the info on gateways directly from system, but gateway names are not there. So they are extracted from possibly non-applied config instead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants