View Issue Details

IDProjectCategoryView StatusLast Update
0002065GNUnetexit daemonpublic2012-02-28 11:05
ReporterChristian Grothoff Assigned ToChristian Grothoff  
PriorityhighSeverityfeatureReproducibilityN/A
Status closedResolutionfixed 
Product VersionGit master 
Target Version0.9.2Fixed in Version0.9.2 
Summary0002065: ICMP support should be added
DescriptionICMPv6 is likely to be particularly critical for routing IPv6 traffic nicely (MTU-discovery!).
TagsNo tags attached.

Relationships

related to 0002064 closedChristian Grothoff new VPN service (working with new exit/dns services) needs to be fully implemented and tested 

Activities

Christian Grothoff

2012-01-21 00:19

manager   ~0005321

TUN library now has ICMP support.

Christian Grothoff

2012-01-21 00:55

manager   ~0005322

Exit daemon has ICMP support as of 19295. However, we might still want to restrict which ICMP types the exit will allow out to the Internet/Service and the mesh-message format might still change (or the data sent might be subject to additional sanitation). See FIXME's in the code.

Christian Grothoff

2012-01-21 16:13

manager   ~0005326

VPN now has rudimentary ICMP support. Code to translate ports/IP addresses in ICMP payload is missing. Also, there is no story yet for translating between ICMPv4 to ICMPv6 and back (that'll be a bit more funky stuff).

Christian Grothoff

2012-01-21 20:05

manager   ~0005334

Documented how ICMP PT should work with the messages. Also specified where we leave out the ICMP payload as it is useless to the other side due to NAT (the exit/vpn will have to make up the full IP+8-bytes of payload anyway for certain message types).

Christian Grothoff

2012-01-21 20:05

manager   ~0005335

Exit daemon now properly discards ICMP payload for messages from TUN to MESH.

Christian Grothoff

2012-01-21 20:33

manager   ~0005336

Last edited: 2012-01-21 23:31

VPN service now properly discards ICMP payloads for messages from TUN to MESH. VPN also considers PT issues and modifies ICMP types from ICMPv4 to ICMPv6 as necessary (SVN 19302).

Open issues include:
1) Synthesis of ICMP payload by exit for certain message types (done in SVN 19307)
2) Synthesis of ICMP payload by vpn for certain message types (done in SVN 19305)
3) ICMP protocol translation by exit (to service) towards TUN (done in SVN 19307)
4) ICMP protocol translation by vpn towards TUN (done in SVN 19304)
5) Might want to move ICMP PT code to TUN library...

Christian Grothoff

2012-01-21 23:32

manager   ~0005339

Declaring victory for now; (5) above does not seem very feasible or critical at this point.

Issue History

Date Modified Username Field Change
2012-01-14 21:51 Christian Grothoff New Issue
2012-01-15 21:45 Christian Grothoff Assigned To => Christian Grothoff
2012-01-15 21:45 Christian Grothoff Status new => assigned
2012-01-17 19:52 Christian Grothoff Relationship added related to 0002064
2012-01-17 19:55 Christian Grothoff Priority normal => high
2012-01-18 23:17 Christian Grothoff Target Version 0.9.3 => 0.9.2
2012-01-21 00:19 Christian Grothoff Note Added: 0005321
2012-01-21 00:55 Christian Grothoff Note Added: 0005322
2012-01-21 00:55 Christian Grothoff Category exit daemon => VPN service
2012-01-21 16:13 Christian Grothoff Note Added: 0005326
2012-01-21 20:05 Christian Grothoff Note Added: 0005334
2012-01-21 20:05 Christian Grothoff Note Added: 0005335
2012-01-21 20:33 Christian Grothoff Note Added: 0005336
2012-01-21 21:12 Christian Grothoff Note Edited: 0005336
2012-01-21 21:38 Christian Grothoff Category VPN service => exit daemon
2012-01-21 23:31 Christian Grothoff Note Edited: 0005336
2012-01-21 23:32 Christian Grothoff Note Added: 0005339
2012-01-21 23:32 Christian Grothoff Status assigned => resolved
2012-01-21 23:32 Christian Grothoff Fixed in Version => 0.9.2
2012-01-21 23:32 Christian Grothoff Resolution open => fixed
2012-02-28 11:05 Christian Grothoff Status resolved => closed