Fixed the server to check correctly number of connections per
[silc.git] / TODO
1 TODO/bugs in Irssi SILC client
2 ==============================
3
4  o Fix the silc_channels_join to parse the command like, with fe.
5    silc_parse_command_line, because currently it ignores all options,
6    including passphrase which makes autojoin impossible to +a channels.
7    Other important options are ignored too.
8
9  o Add local command to switch the channel's private key when channel has
10    several private keys.  Currently sending channel messages with many
11    keys is not possible because changing the key is not possible by the
12    user.
13
14  o /cumode for unknown nick does not give any error message.
15
16
17 TODO/bugs In SILC Client Library
18 ================================
19
20  o The PRIVATE_MESSAGE_KEY packet is not handled (it is implemented 
21    though).  This should be added and perhaps new client operation
22    should be added to notify application that it was received and
23    set the key only if application wishes to set (accept the key) it.
24
25  o Additions to do after protocol version 1.1:
26
27         o Fix the NICK_CHANGE notify handling not to create new entry
28           for the changed client, but take the nickname from the notify
29           (removes need for resolving as well).  Protocol TODO entry 3.
30
31         o Add support for list of errors in command replies.  Protocol
32           TODO entry 1.
33
34
35 TODO/bugs In SILC Server
36 ========================
37
38  o Configuration file additions:
39
40         o Add version handling, to allow, disallow certain versions to
41           connect.
42
43         o Add incoming connection frequency, incoming connection frequency
44           for single IP address, key exchange frequency, key exchange
45           frequency for single IP. Add also frequency base.
46
47         o Add hashed passwords to config file.
48
49         o Add rehashing support.
50
51  o If server send CUMODE_CHANGE notify (like setting founder) to router
52    and router does not have founder on channel (founder is left or there's
53    no founder on channel at all), the router will accept the server's
54    founder mode change, even though it perhaps should not do that.
55
56  o The router should check for validity of received notify packets from
57    servers (after all buggy servers may send notify that is actually
58    something that should have not been sent).
59
60  o Add a timeout to handling incoming JOIN commands.  It should be 
61    enforced that JOIN command is executed only once in a second or two
62    seconds.  Now it is possible to accept n incoming JOIN commands
63    and process them without any timeouts.  THis must be employed because
64    each JOIN command will create and distribute the new channel key
65    to everybody on the channel.
66
67  o Backup router related issues
68
69         o Channel user mode changes are notified unnecessarely when
70           switching to backup router on router crash.
71
72  o Lots of statistics updating is missing around the server.
73
74  o If client's public key is saved in the server (and doing public key
75    authentication) then the hostname and the username information could
76    be taken from the public key.  Should be a configuration option!
77
78
79 TODO/bugs In SILC Libraries
80 ===========================
81
82  o Fix possible buffer overflows in silc_id_render function.
83
84  o WIN32 silc_net_create_connection_async does not work the same way
85    than on Unix.  Do it with threads on WIN32.  The function works but
86    is not actually async currently.
87
88  o Rewrite the lib/silcsim/silcsim.h.  The SilcSimContext should be
89    private and silc_sim_alloc should take necessary arguments.
90
91  o SILC RNG does not implement random seed files, and they should be
92    implemented.
93
94  o The SilcSocketConnection in the SFTP interface is actually redundant
95    and should perhaps be removed.  The application can save it in the
96    context it provides, which is delivered by SFTP libary to all 
97    callback functions.
98
99  o EPOC specific additions/changes required:
100
101         o lib/silcutil/epoc routines missing or not completed.
102
103         o Rewrite the lib/silcutil/silcprotocol.[ch] not to have 
104           [un]register functions, but to make it context based all the 
105           way.  The alloc should take as argument the protocol type and 
106           its callback (not only final callback).  It is not good that we 
107           have now global list of registered protocols.
108
109         o Global RNG should not be allowed on EPOC, since it won't
110           work.  Every function in library that calls the global RNG must
111           also take the RNG context as argument so that application can
112           provide it if it doesn't want to use global RNG.
113
114         o Think some solution to the global crypto lists in lib/silccrypt.
115           They won't work on EPOC.
116
117         o Something needs to be thought to the logging globals as well, 
118           like silc_debug etc.  They won't work on EPOC.  Perhaps logging
119           and debugging is to be disabled on EPOC.
120
121
122 TODO in Toolkit Documentation
123 =============================
124
125 Stuff that needs to be done in order to complete the Tooolkit Reference
126 Manual.
127
128  o Lots of ROBOdoc header formatting is undone in lib/silcutil, and 
129    lib/silccrypt.
130
131  o Write "Programming with Toolkit" document, describing how to build
132    Toolkit, how the build system works, where is everything, how
133    new (external) projects can be glued into Toolkit (use irssi as an
134    example), and how external projects can use Toolkit without gluing into
135    it (how to link etc), debugging, architecture, types, etc.
136
137  o Write "Platform Implementations" document to describe what platforms
138    Toolkit support, what has been implemented, what has not been, what
139    wors differently etc.
140
141
142 TODO in SILC Protocol
143 =====================
144
145 Current protocol version is 1.0.  However, it is far from being perfect,
146 and needs to include additional features.  Following protocol TODO entries
147 describe new stuff to be added to protocol versions 1.x.
148
149  1. Re-define the Status Payload: it is now 16 bits, split it into two
150     8 bits fields.  First field includes status types from 0 - 9 and
151     10 - n *if* it is not an list of errors.  If it is list of errors then
152     the first field includes 1, 2 and/or 3, and the second field includes
153     the error status 10 - n.  This way it is possible to send multiple
154     errors (list of errors) and we have a way to tell the receiver that
155     there will be other errors as well.  The second field is used only
156     if there is list of errors.  If normal status, or normal (single)
157     error status the second field is set to zero, and must be ignored.
158     Hence, the status works same way as now except for list of errors.
159     To be included in protocol version 1.1.
160
161  2. Define that WHOIS and IDENTIFY commands must send list of errors
162     if multiple Client ID (or Channel ID and Server ID for IDENTIFY) was
163     requested and was not found.  Each unfound entry must cause an error
164     command reply to the sender.  Also define that errors must be sent
165     *after* sending successfully found entries (this way receiver may
166     ignore them).  To be included in protocol version 1.1.
167
168  3. Define the NICK_CHANGE notify to send the changed nickname as a new
169     third argument.  This will make the NICK_CHANGE notify handling easier
170     in the receiver's end (client primarily) since it removes the
171     requirement that receiver must resolve (using IDENTIFY or WHOIS) the
172     new Client ID received in the notify (because of the new nickname is
173     unknown).  To be included in protocol version 1.1.
174
175  4. Add "request parameters" or similar to the WHOIS command, which can
176     be used to request various parameters (something not returned by
177     standard WHOIS command) about clients (info that could be fetched
178     even from clients).  Additional specification (or appendix) should 
179     be done to define the payload and the parameters.  It could be used
180     to make the WHOIS command support various search conditions as well.
181     This would be the way to extend the WHOIS command to support various
182     new features without always making the command incompatible to previous
183     version.  To be included in protocol version 1.1.
184
185  5. Inviting and banning by public key should be made possible.  To be
186     included in protocol version 1.x.
187
188  6. Add perhaps SILENCE_USERS, SILENCE_OPERS channel user modes which
189     can be used to silence (moderate) normal users and opers (this set
190     only by founder).  To be included in protocol version 1.1.
191
192  7. Channel Message Payload needs slight redesining to include the IV
193     field to the MAC generation of the payload.  It is authenticated
194     by the packet's MAC but not by the payload's MAC.  Since the IV
195     belongs to the payload, its integrity should be protected by the
196     payload MAC and not alone by packet MAC.  To be included in protocol 
197     version 1.1.
198
199  8. Remove the administrative commands from the protocol all together.
200     It does not make sense for the protocol to define how a server is
201     reconnected or shutdown, since they are implementation and 
202     configuration issues.  Besides protocol provides only limited set of
203     administrative commands and cannot define all that one could imagine.
204     To be included in protocol version 1.1.
205
206  9. Add SILC_MESAGE_FLAG_REPLY for being other side to the
207     SILC_MESSAGE_FLAG_REQUEST.  Add generic SILC_MESSAGE_FLAG_DATA, which
208     can include generic payload, which can include generic data.  The
209     payload definition is left out for now.  To be included in protocol
210     version 1.1.
211
212  10. Check command reply error status types in various commands,
213      specifically NO_FOPRIV is missing from many commands.  To be 
214      included in protocol version 1.1.
215
216  11. Change the wording in Private Message Key Payload definition to
217      describe the problems of trusting the payload, and to indicate that
218      the receiver may not accept the key in the payload, and to describe
219      other means of distributing a key.
220
221  12. The router to router connection diagram in spec-xx is showing the
222      primary routers direction to wrong direction.  Swap it.