Ticket #3 (closed enhancement: fixed)

Opened 5 years ago

Last modified 5 years ago

Cleanup dns code

Reported by: yarrick Owned by: flex
Priority: major Milestone: 0.4.0 "Run Home"
Version: 0.3.4 Keywords:
Cc:

Description (last modified by yarrick) (diff)

All socket reads/writes (recv_from and send_to) should be done in iodine[d].c, to make testing easier and get better structure.

dns_settarget should be removed as well. Also look at dns_send_chunk, dns_ping, dns_write, dns_login and dns_send_version.

Change History

comment:1 Changed 5 years ago by yarrick

  • Owner set to flex

comment:2 Changed 5 years ago by yarrick

Moved dnsd_read to iodined.c in [404]

comment:3 Changed 5 years ago by yarrick

  • Description modified (diff)

comment:4 Changed 5 years ago by flex

[408] did some major cleanup by moving all client-code from dns.c to iodine.c only trace of network left in dns.c is settarget, used by decodedata, used by the server

comment:5 Changed 5 years ago by yarrick

Commited [409], now the test cases compile. They still need to be adapted for the test cases to run successfully.

comment:6 Changed 5 years ago by yarrick

Commited [411], now basic tunneling works again. Left to do is some renaming in iodine.c (like dns_read method), to get rid of dnsd_send in dns.c, and to fix test cases.

comment:7 Changed 5 years ago by yarrick

Renamed functions in iodine.c: [413]

comment:8 Changed 5 years ago by yarrick

Moved dnsd_send out of dns.c [415]

comment:9 Changed 5 years ago by yarrick

Alert! We broke lots of stuff while doing this. Dont expect big packets to get sent upstream correctly just yet...

comment:10 Changed 5 years ago by yarrick

The breakage is back to normal now :)

comment:11 Changed 5 years ago by yarrick

  • Status changed from new to closed
  • Resolution set to fixed

After small code adjustments [421] [422] [423], the testcases now run fine again.

Note: See TracTickets for help on using tickets.