1 This file lists minor work items (also possibly called "known bugs").
2 We are not tracking them in Mantis yet since there are too many and no
3 sane end-user should care about this codebase yet anyway.
7 - only connect() sockets that are ready (select()) [Nils]
8 [On W32, we need to select after calling socket before
11 - implement transport API to pretty-print transport address
12 + transport_api extension (API extension!)
13 + service-transport extension (protocol extension)
14 - add calls to statistics in various places
15 - implement gnunet-transport (transport configurator / tester)
16 - UPnP-based IP detection
17 (Note: build library always, build service when libxml2/etc. are available)
18 - instantly filter addresses from *other* peers that
19 are *equal* to our own address + port (i.e., localhost:2086). We
20 no longer filter those for outgoing (helps with loopback testing
21 and keeps the code clean), but we should filter strictly *impossible*
22 incoming addresses! This is for efficiency, not correctness.
23 - We currently are happy to take any address told to us in a WELCOME
24 to our set of addresses; we should have some minimal threshold-based
25 scheme, limiting both the total number of addresses that we accept
26 this way as well as requiring multiple confirmations; also, we
27 should possibly try to confirm that the given address works for
28 us ourselves (loopback-style) before adding it to the list
30 + we may be able to simplify WELCOME messages (no need to add
31 addresses there anymore, but may help to learn them there anyway...).
32 + we probably want some kind of voting/counting for learning IP addresses
33 (maybe including IP addresses in ads proportional to how often others
34 report them? we at least need some protection against >64k HELLOs!),
35 + provide a way to give the user a list of "learned" IP addresses and
36 a way to easily "veto" addresses off the list!
37 => If MiM attacker uses vetoed address, blacklist the specific IP for
38 the presumed neighbour!
39 - [./transport/gnunet-service-transport.c:173]: (style) struct or union member 'TransportPlugin::rebuild' is never used
40 - [./transport/plugin_transport_tcp.c:391]: (style) struct or union member 'Plugin::address_update_task' is never used
42 - [./fs/gnunet-service-fs.c:208]: (style) struct or union member 'LocalGetContext::results_bf_size' is never used
43 - [./fs/gnunet-service-fs.c:501]: (style) struct or union member 'PendingRequest::used_pids_size' is never used
44 - [./fs/gnunet-service-fs.c:654]: (style) struct or union member 'ConnectedPeer::last_client_replies' is never used
45 - [./fs/gnunet-service-fs.c:669]: (style) struct or union member 'ConnectedPeer::avg_delay' is never used
46 - [./fs/gnunet-service-fs.c:675]: (style) struct or union member 'ConnectedPeer::avg_priority' is never used
47 - [./fs/gnunet-service-fs.c:688]: (style) struct or union member 'ConnectedPeer::pending_requests' is never used
48 - [./fs/gnunet-service-fs.c:694]: (style) struct or union member 'ConnectedPeer::last_p2p_replies_woff' is never used
49 - [./fs/gnunet-service-fs.c:700]: (style) struct or union member 'ConnectedPeer::last_client_replies_woff' is never used
51 - If the topology daemon crashes, peers that were put on the
52 blacklist with transport will never be removed from it (until
53 transport service dies); we should use the blacklist notification
54 API to learn about the exact set of blacklisted peers at all times
55 (FIXME: the transport_api implementation of blacklisting
56 also does not work nicely for this since it won't let us know about
57 disconnect-reconnect events and the implicit whitelisting
58 that might happen here; that's not so bad since we will
59 re-blacklist on pre-connect attempts anyway, so this is
61 - the code uses the term 'blacklist' for both peers that are forbidden
62 to connect (i.e. F2F mode) as well as peers that we currently
63 won't try to actively connect to ourselves (since we just tried);
64 This is confusing. We need two distinct terms.
65 - move code to use hash table instead of linked list
66 - instead of periodically discarding blacklisted entries,
67 simply add task that is triggered at the right time (earlier free,
69 - check if new HELLO learned is different from old HELLO
70 before resetting entire state!
72 - auto-generate "defaults.conf" using gnunet-setup from "config.scm"
73 - integrate all options into "config.scm"
74 - change config-file writing to exclude options set to default values
76 - better tracking of which config changes actually need to cause process restarts by ARM.
77 - listen for requests to discover dependencies between services (and avoid
78 having to explicitly program start requests)
79 - better crash management (attach debugging support, capture and analyze
80 debug output, detect random vs. deterministic crashes)
83 - Better SSL-support for MHD
86 - active reply route caching design & implementation of service,
89 - consider changing API for peer-group termination to
90 call continuation when done
93 - 'server' uses 'GNUNET_PEERINFO_iterate', should probably switch to notification API
94 (for more instant / up-to-date hostlists at lower cost) [OPTIMIZATION]