Skip to content

added encapsulation attributes for Iptun and Sittun to support SIT tunnels#281

Merged
vishvananda merged 1 commit intovishvananda:masterfrom
skoef:iptunEncap
Oct 26, 2017
Merged

added encapsulation attributes for Iptun and Sittun to support SIT tunnels#281
vishvananda merged 1 commit intovishvananda:masterfrom
skoef:iptunEncap

Conversation

@skoef
Copy link
Contributor

@skoef skoef commented Sep 27, 2017

The Iptun struct didn't support encapsulation attributes like source or destination port yet, while Gretap did and the feature appeared for both interface types around the same time. I basically copied the parsing and adding of the attributes from Gretap to Iptun and it seems to work fine.

@skoef
Copy link
Contributor Author

skoef commented Oct 5, 2017

For this project I'm doing I would like SIT tunnel support in this library as well. Since these links are similar to Iptun I could ofcourse copy/paste relevant functions and type of Iptun but I'm not sure that is the way to go here

@skoef skoef changed the title added encapsulation attributes to Iptun added encapsulation attributes for Iptun and Sittun to support SIT tunnels Oct 11, 2017
@vishvananda
Copy link
Owner

This looks reasonable. Can you squash it down into a single commit?

Thanks

added encapsulation attributes to Gretun
implemented Sittun struct for supporting SIT tunnels
@skoef
Copy link
Contributor Author

skoef commented Oct 20, 2017

I thought github featured a squash&merge button, nevertheless: it's squashed, no problem!

@vishvananda
Copy link
Owner

it does support squash and merge, but I prefer rebase and merge to keep a linear commit tree. :)

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.

2 participants