Skip to content

Add support for LWTUNNEL_ENCAP_IP6 #148

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Toorero
Copy link

@Toorero Toorero commented Mar 16, 2025

This should fix #144

@Toorero Toorero force-pushed the feature/lwtunnel_ip6 branch from b64356a to 04ecc20 Compare March 16, 2025 23:32
@cathay4t cathay4t force-pushed the feature/lwtunnel_ip6 branch from 04ecc20 to cfdb3be Compare April 30, 2025 09:06
@Toorero Toorero force-pushed the feature/lwtunnel_ip6 branch from cfdb3be to f93a3bd Compare May 19, 2025 08:59
@Toorero
Copy link
Author

Toorero commented May 19, 2025

Rebased and requested changes addressed.

))
}
_ => {
return Err(DecodeError::from(
Copy link
Member

@cathay4t cathay4t Jun 16, 2025

Choose a reason for hiding this comment

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

Instead of return error on unknown LWTUNNEL_IP6_XXX, please use RouteIp6Tunnel::Other to store it. We have many existing code doing that, just copy them.

@cathay4t cathay4t added the Wait_Submitter PR reviewed with change requests label Jun 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Wait_Submitter PR reviewed with change requests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missing Bindings for LWTUNNEL_ENCAP_IP6
2 participants