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

Doesn't NAT ICMP packets #15

Closed
talex5 opened this issue Dec 30, 2015 · 4 comments
Closed

Doesn't NAT ICMP packets #15

talex5 opened this issue Dec 30, 2015 · 4 comments

Comments

@talex5
Copy link
Contributor

talex5 commented Dec 30, 2015

Pings don't need to survive the NAT process. Is there any support for something like https://tools.ietf.org/html/rfc5508 ?

@yomimono
Copy link
Contributor

Not sure what you mean by "pings don't need to survive the NAT process" - can you elaborate?

"Is there any support" - nope, but there should be. Thanks for bringing this RFC to my attention. Splitting the ICMP code out of mirage-tcpip's IPv4 module, like we did for ARP, would make handling ICMP as it specifies easier. As far as I can tell it should be possible, albeit potentially a bit redundant, to implement at least the error packet section within the current design of both mirage-tcpip and mirage-nat, though.

Is there a specific subset of the spec you need?

@talex5
Copy link
Contributor Author

talex5 commented Dec 30, 2015

Sorry, typo; I meant "don't seem to survive".

I don't have any particular requirements - the more that works the better :-) It's for https://github.com/talex5/qubes-mirage-firewall (and any comments or suggestions about my use of mirage-nat are welcome!)

@talex5
Copy link
Contributor Author

talex5 commented Mar 10, 2017

(this is implemented in #20)

@yomimono
Copy link
Contributor

Thanks @talex5!

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

No branches or pull requests

2 participants