View Issue Details

IDProjectCategoryView StatusLast Update
0002065GNUnetexit daemonpublic2012-02-28 10:05
ReporterChristian Grothoff Assigned ToChristian Grothoff Due Date
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-20 23:19

manager   ~0005321

TUN library now has ICMP support.

Christian Grothoff

2012-01-20 23: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 15: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 19: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 19:05

manager   ~0005335

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

Christian Grothoff

2012-01-21 19:33

manager   ~0005336

Last edited: 2012-01-21 22: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 22:32

manager   ~0005339

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

Issue History

Date ModifiedUsernameFieldChange
2012-01-14 20:51Christian GrothoffNew Issue
2012-01-15 20:45Christian GrothoffAssigned To => Christian Grothoff
2012-01-15 20:45Christian GrothoffStatusnew => assigned
2012-01-17 18:52Christian GrothoffRelationship addedrelated to 0002064
2012-01-17 18:55Christian GrothoffPrioritynormal => high
2012-01-18 22:17Christian GrothoffTarget Version0.9.3 => 0.9.2
2012-01-20 23:19Christian GrothoffNote Added: 0005321
2012-01-20 23:55Christian GrothoffNote Added: 0005322
2012-01-20 23:55Christian GrothoffCategoryexit daemon => VPN service
2012-01-21 15:13Christian GrothoffNote Added: 0005326
2012-01-21 19:05Christian GrothoffNote Added: 0005334
2012-01-21 19:05Christian GrothoffNote Added: 0005335
2012-01-21 19:33Christian GrothoffNote Added: 0005336
2012-01-21 20:12Christian GrothoffNote Edited: 0005336
2012-01-21 20:38Christian GrothoffCategoryVPN service => exit daemon
2012-01-21 22:31Christian GrothoffNote Edited: 0005336
2012-01-21 22:32Christian GrothoffNote Added: 0005339
2012-01-21 22:32Christian GrothoffStatusassigned => resolved
2012-01-21 22:32Christian GrothoffFixed in Version => 0.9.2
2012-01-21 22:32Christian GrothoffResolutionopen => fixed
2012-02-28 10:05Christian GrothoffStatusresolved => closed