e52c265c2853b1e7d8e06d1d49e5d6f4febccbc1
[silc.git] / CHANGES
1 Thu Feb  1 23:31:21 EET 2001  Pekka Riikonen <priikone@poseidon.pspt.fi>
2
3         * Fixed channel ID decoding in server's JOIN command reply in
4           silcd/command_reply.c
5
6         * Fixed braodcasting of replace ID payload to not to send it if
7           we are standalone server in silcd/packet_receive.c.
8
9         * Fixed all channel message sending routines to not to send
10           packets to clients that has router set, since they are routed
11           separately in the same function earlier.  Affects file
12           silcd/packet_send.c and all channel packet sending functions.
13
14 Thu Feb  1 21:32:27 EET 2001  Pekka Riikonen <priikone@poseidon.pspt.fi>
15
16         * Fixed some minor bugs in client when sending WHOIS command.  The
17           arguments was in wrong order.
18
19         * Removed statis function add_to_channel from server in 
20           silcd/command.c that was previously used with the joining but
21           is obsolete now.
22
23         * Tested USERS command in router environment successfully with two
24           routers, two servers and two clients.
25
26 Thu Feb  1 00:54:26 EET 2001  Pekka Riikonen <priikone@poseidon.pspt.fi>
27
28         * Reorganized the USERS command and command reply in client library
29           in lib/silcclient/command.c and lib/silcclient/command_reply.c.
30           When the command is given by user we register a pending command
31           callback that will reprocess the command after the reply has been
32           received from the server.  When reprocessing the packet we then
33           display the information.  Thus, the USERS information is displayed
34           now in the command callback instead of in the command reply
35           callback.  The processing of the command is same as previously
36           when server has sent the command reply in the JOINing process.
37
38         * Added to USERS command in silcd/command_reply.c to join the client,
39           we didn't use to know about, to the channel after we've created
40           a client entry for it.  Also, for clienet we did know already still
41           check whether it is on the channel or not and add it if not.
42
43         * Removed silc_server_command_join_notify as the function and its
44           use was obsolete.
45
46 Tue Jan 30 22:39:15 EET 2001  Pekka Riikonen <priikone@poseidon.pspt.fi>
47
48         * Changed the client's pending command handling to the same as the
49           server's pending command handling.  It is also now possible to
50           execute command reply functions from other command reply
51           function as the function callbacks for commands and command
52           replies are one and same.  The pending commands are not static
53           list anymore, it is mallocated SilcDList in lib/silcclient/client.h
54           in client connection context.  Thus, pending commands are server
55           connection specific as it is convenient.
56
57           Changed the function silc_client_command_pending and
58           silc_client_command_pending_del and added new function
59           silc_client_command_pending_check.  Removed the 
60           SILC_CLIENT_CMD_REPLY_EXEC, and SILC_CLIENT_PENDING_COMMAND_CHECK
61           macros.
62
63         * Added cmd_ident, current command identifier, to the client
64           connection context in lib/silcclient/client.h to keep track on
65           command identifiers used in command sending.  Client's command reply
66           function handling now supports the mandatory command identifiers.
67
68         * Added SILC_CLIENT_COMMAND_EXEC_PENDING macros to all command reply
69           funtions in client to fully support pending command callbacks.
70
71         * NOTE: the name_list in USERS (old NAMES) command is NOT sent anymore
72           as one of the arguments to the application in the command reply
73           client operation.
74
75         * NOTE: The FORWARDED flag is depracated.  It used to be depracated
76           before first releasing SILC but came back.  Now it is removed again
77           and should come back nomore.  The FORWARDED flag was used only
78           by the JOINing procedure by forwarding the command packet to router.
79           Now, the JOINing procedure has been changed to more generic (due
80           to various router environment issues) and FORWARDED is not needed
81           anymore for anything.  The protocol specification is yet to be
82           updated.
83
84           Now, removed silc_server_packet_forward from server and the flag
85           SILC_PACKET_FORWARDED from lib/silccore/silcpacket.h.
86
87 Tue Jan 30 00:05:05 EET 2001  Pekka Riikonen <priikone@poseidon.pspt.fi>
88
89         * Renamed NAMES command to USERS command.  The NAMES was named that
90           due to historical reasons.  Now it is renamed.  Also, rewrote
91           parts of the USERS command.  The nickname list is not sent anymore
92           by the server.  Only Client ID and mode lists are sent in the USERS
93           command.  Changed this also to the protocol specification.
94
95           The client now resolves the names and stuff after it receives
96           the USERS list from the server when joining to the channel.
97
98         * WHOIS and IDENTIFY commands has been changed to support multiple
99           Client ID's per command.  One can now search for multiple users
100           in the network by sending only one WHOIS or IDENTIFY command.
101           Changed the code and the protocol specifications.
102
103         * Removed silc_server_command_identify_parse and changed that IDENTIFY
104           uses silc_server_command_whois_parse to parse the request. */
105
106         * If normal server, do not parse the WHOIS and IDENTIFY requests
107           before sending it to the router.  Saves some time.
108
109 Sun Jan 28 16:19:49 EET 2001  Pekka Riikonen <priikone@poseidon.pspt.fi>
110
111         * Fixed JOIN command on client library.  Wrong number of arguments
112           used to crash the client.
113
114         * Added silc_server_channel_has_global function to check whether
115           channel has global users or not.
116
117         * Added silc_server_channel_has_local function to check whether channel
118           has locally connected clients on the channel.
119
120         * The silc_server_remove_from_one_channel now checks whether the
121           channel has global users or not after given client was removed from
122           the channel.  It also checks whether the channel has local clients
123           on the channel anymore.  If it does not have then the channel entry
124           is removed as it is not needed anymore.
125
126         * The silc_server_notify now checks on JOIN notify whether the joining
127           client is one of locally connected or global.  If it is global then
128           the channel has now global users on the channel and that is marked
129           to the channel entry.  Also, it now saves the global client to
130           global list who is joining and JOINs it to the channel.  This is
131           for normal server, that is.
132
133           Changed silc_server_send_notify_on_channel, 
134           silc_server_packet_relay_to_channel and 
135           silc_server_packet_send_to_channel check if we are normal server
136           and client has router set (ie. global client) do not send the
137           message to that client, as it is already routed to our router.
138
139         * Implemented LEAVE notify type handling in silc_server_notify 
140           function.
141
142         * Tested LEAVE command in router environment successfully.  Tested
143           with two routers, two servers and two clients.
144
145         * Updated TODO.
146
147         * idlist_find_xxx_by_id routines now dumps the ID on the debug mode.
148
149         * Implemented SIGNOFF notify type handling in silc_server_notify
150           function.
151
152         * silc_server_remove_id now removes the client entry from all channels
153           it has joined and thusly sends SIGNOFF notify type.
154
155         * Rewrote the NAMES list generation in server by removing two excess
156           loops.  The lists are created now inside one loop.
157
158 Sat Jan 27 22:34:56 EET 2001  Pekka Riikonen <priikone@poseidon.pspt.fi>
159
160         * silc_server_remove_channel_user checks now also global list
161           for channel and client.
162
163         * silc_server_new_channel_user checks now both local and global
164           list for channel and client.  Fixed a bug in client id decoding.
165           Used to decode wrong buffer.
166
167         * silc_server_channel_message checks now both local and global
168           list for channel entry.
169
170         * Tested channel joining (hence JOIN) in router environment
171           successfully.  Tested with two routers, two servers and two
172           clients.
173
174         * Tested channel message sending in router environment successfully.
175
176 Thu Jan 11 03:22:57 EET 2001  Pekka Riikonen <priikone@poseidon.pspt.fi>
177
178         * Added silc_server_save_channel_key into server.[ch] to save the
179           received channel key in Channel Key payload processing. It is
180           also used in JOIN command reply handling.
181
182           Equivalent function silc_client_save_channel_key added into
183           client.[ch] into client library.
184
185         * Changed JOIN command reply to send information whether the channel
186           was created or not (is existing already) and the channel key 
187           payload.  Changed protocol specs accordingly.
188
189         * Fixed bugs in WHOIS and IDENTIFY command reply sending when
190           the request was sent by ID and not by nickname.  Crashed on
191           NULL dereference.
192
193 Sat Dec 23 21:55:07 EET 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
194
195         * Fixed a bug in Client library.  IDENTIFY and WHOIS reply functions
196           now correctly save the received data.
197
198         * silc_server_free_sock_user_data now notifies routers in the 
199           network about entities leaving the network.
200
201           At the same time implemented functions silc_server_remove_id
202           and silc_server_send_remove_id to receive and send REMOVE_ID
203           packets.  The packet is used to notify routers in the network
204           about leaving entities.  The ID removed will become invalid in
205           the network.
206
207         * Added function silc_idlist_del_server into server. Removes and
208           free's server entry from ID list.
209
210         * silc_server_private_message function now checks, if we are router,
211           that the destination ID really is valid ID, naturally.
212
213         * In router when NEW_ID packet is received (for new client) the
214           hash of the Client ID is saved in the ID Cache but the
215           client->nickname is set to NULL, instead of putting the hash
216           to it as well.
217
218           IDENTIFY command now also checks that client->nickname must be
219           valid. If it is not if will request the data from the server who
220           owns the client.  Added new function 
221           silc_server_command_identify_check.
222
223         * Added silc_command_set_command into lib/silccore/silcommand.[ch]
224           to set the command to already allocated Command Payload.
225
226         * Tested private message sending in router environment with two
227           routers, two servers and two clients.  Fixed minor bugs and now
228           it works fine.
229
230         * Fixed segfault from client's NAMES command. Used to crash if
231           not on any channel.
232
233         * Forwarded packets must not be routed even if it is not destined
234           to the receiver.  Changed server code comply with this.
235
236 Sun Dec 17 14:40:08 EET 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
237
238         * Added `require_reverse_mapping' boolean value to ServerParams
239           structure. If TRUE (not default) the server will require that
240           the connecting host has fully qualified domain name.
241
242           If the reverse mapping is not required and hostname could not be
243           found the IP address is used as hostname.
244
245 Sat Dec 16 17:39:54 EET 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
246
247         * Implemented version string checking to both client and server.
248           The check is incomplete currently due to the abnormal version 
249           strings used in development version of SILC.
250
251         * Changed all command functions in server to use the new
252           CHECK_ARGS macro.
253
254 Fri Dec 15 15:55:12 EET 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
255
256         * Changed char *data to unsigned char *data in ID Cache system to
257           support binary data as ID Cache data. Changed code to support
258           binary data in lib/silccore/idcache.c.
259
260         * Renamed silc_server_packet_relay_command_reply to 
261           silc_server_command_reply as it is normal packet receiving
262           function. Rewrote the function to accept command replys for
263           servers and not only for clients.
264
265         * Mark remote router always as registered server if we are connecting
266           to it.  Otherwise, commands sent by the router to us are ignored.
267
268         * All ID List find routines now returns the ID Cache Entry pointer
269           as well if requested.
270
271         * WHOIS command works now in router environment, tested with two
272           routers, two servers and two clients.
273
274         * Cleaned up and rewrote IDENTIFY command. IDENTIFY should work now
275           in router environment (as it is almost equivalent to WHOIS) but
276           hasn't been tested thoroughly.  Added new functions:
277
278           silc_server_command_identify_parse
279           silc_server_command_identify_send_reply
280           silc_server_command_identify_from_client
281           silc_server_command_identify_from_server
282
283         * Disabled route cache adding because adding two different ID's with
284           same IP replaces the old cache entry thus giving wrong route.
285           The entry->router->connection is always the fastest route anyway
286           so route cache may not be needed.  Of course, new routes maybe
287           established after receiving the ID when the entry->router->connection
288           might not be anymore the most optimal.
289
290 Thu Dec 14 15:55:35 EET 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
291
292         * Add route cache for received ID for fast routing.
293
294         * Added silc_server_packet_route to route received packet on router
295           that is not destined to us.
296
297         * Renamed silc_server_get_route to silc_server_route_get.
298
299         * Added id_string and id_string_len fields into SilcServer to
300           include encoded ServerID for fast comparing without excess
301           encoding of the ID's.
302
303         * Cleaned up WHOIS command on server side. Added following static
304           functions:
305
306           silc_server_command_whois_parse
307           silc_server_command_whois_check
308           silc_server_command_whois_send_reply
309           silc_server_command_whois_from_client
310           silc_server_command_whois_from_server
311
312         * Added macro SILC_SERVER_COMMAND_CHECK_ARGC to check mandatory
313           arguments in command replies. All command functions should be
314           updated to use this macro.
315
316 Sun Dec 10 23:52:00 EET 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
317
318         * Minor typo fixes on command reply handling on server.
319
320 Tue Nov 28 11:05:39 EET 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
321
322         * Added silc_server_command_add_to_channel internal routine to add
323           the client to the channel after router has created the channel and
324           sent command reply to the server.
325
326         * Added generic silc_server_send_command to send any command from
327           server.
328
329         * Use static buffer with ID rendering instead of duplicating data.
330
331 Mon Nov 27 21:39:40 EET 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
332
333         * Fixed a channel user mode bug when joining to a channel server gave
334           everybody channel founder rights, oops.
335
336         * We mark ourselves as the router of the incoming server connection
337           if we are router ourselves.  This way we can check in some packet
338           sending functions whether it is locally connected server.  For
339           incoming router connections we put NULL.
340
341         * For router sending packets locally means now always sending the
342           packet cell wide; to local clients and local servers.  For normal
343           server sending packet locally means sending it to only local
344           clients.
345
346         * Fixed the JOIN command to really work in router environment.  If the
347           channel is created it is always created by the router.  Router is
348           also responsible of making the initial joining to the channel,
349           sending JOIN notify to the sending server and distributing 
350           NEW_CHANNEL and NEW_CHANNEL_USER packets.  Hence, if the channel
351           does not exist server doesn't do anything else but forward the
352           command to the router which performs everything.
353
354         * Added silc_server_send_channel_key function to send the Channel Key
355           payload.
356
357         * Added silc_server_create_channel_key to create new channel key.  The
358           channel key is now re-generated everytime someone joins or leaves
359           a channel, as protocol dictates.  Note: channel->key_len is the
360           key length in bits.
361
362 Wed Nov 22 22:14:19 EET 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
363
364         * Splitted server.[ch] finally.  Created now packet_send.[ch] and
365           packet_receive.[ch] to separate packet sending and receiving
366           routines.  The server.[ch] now includes everything else including
367           actual packet processing (writing and reading data) and other
368           server issues.
369
370           Renamed silc_server_private_message_send_internal to
371           silc_server_send_private_message.  The routine is still though
372           used only to relay private messages as server does not send
373           private messages itself.
374
375           Renamed silc_server_new_channel to silc_server_create_new_channel
376           and added new function sicl_server_new_channel that handles the
377           incoming New Channel packet.  Added also new sending function
378           silc_server_send_new_channel to send New Channel Payload.
379
380         * Added new function silc_server_notify to process incoming notify
381           packet to the server/router. Server may then relay the notify
382           to clients if needed.
383
384         * Added new function silc_server_new_channel_user to process incoming
385           New Channel User packet.  Router will redistribute the packet and
386           send JOIN notify to its local clients and locally connected servers
387           if needed.  Normal server will send JOIN notify to its local client
388           on same channel when received this packet.  Added also corresponding
389           sending function silc_server_send_new_channel_user to sent the
390           payload.
391
392         * Added boolean route argument to send_notif_to_channel and
393           packet_send_to_channel functions to attempt to route the packet
394           if it is TRUE and send only locally if it is FALSE.
395
396 Tue Nov 21 19:49:31 EET 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
397
398         * silc_server_replace_id now broadcasts the received replace ID
399           packet if it is not broadcast packet already. The router must
400           broadcast to inform other routers about changed ID.
401
402         * Added backpointer to server's router into SilcServer context in
403           silcd/server_internal.h.
404
405         * Fixed silc_server_packet_broadcast to send correct broadcast
406           packets.
407
408         * The channel key is now distributed to the local client as soon
409           as it is received from the router (in router environment) so that
410           no other packet may be sent for the channel until client has 
411           received the key.
412
413         * silc_server_remove_channel_user now broadcasts the received
414           Remove Channel User packet if it is not broadcast packet already.
415           The router must broadcast to inform other routers about removed
416           channel user.
417
418         * Added users field into SilcPacketContext that is a reference count
419           of the context.  One can increase the reference count by calling
420           silc_packet_context_dup which is now changed to just increase the
421           reference count instead of duplicating the data.  The reference
422           count is decresed by calling silc_packet_context_free that will
423           free the data after the reference count hits zero.
424
425           For now on the packet context and everything allocated into it
426           (including the raw packet from network) must be freed by calling
427           the new silc_packet_context_free function.  Added also new function
428           silc_packet_context_alloc that must be used now to allocate the
429           context.  This also means that if a routine is asynchronous from
430           silc_[client/server]_packet_parse_type the packet context must
431           be duplicated by calling silc_packet_context_dup.  Otherwise it
432           gets free'd after silc_[client/server]_packet_parse_type returns.
433           Also, one must remember that if packet is duplicated then its 
434           reference count must be decresed by calling the free function as
435           many times as it was duplicated.
436
437         * Changed SilcBuffer field from protocol contexts to SilcPacketContext
438           from both client and server.
439
440 Mon Nov 20 23:47:03 EET 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
441
442         * Made joining to a channel working in router environment.
443
444         * Cleaned up JOIN command on server side and create function
445           silc_server_command_join_channel internal routine to make the
446           joining happen.
447
448 Thu Nov  9 21:12:39 EET 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
449
450         * Changed silc_command_pending list to SilcDList.  Also, added
451           `ident' field to SilcServerCommandPending structure to identify
452           the reply and to call correct callback.
453
454           Added silc_server_command_pending_check function to replace the
455           corresnponding macro.  The silc_command_pending list is not
456           extern anymore.
457
458         * Added silc_command_set_ident into lib/silccore/silccommand.[ch]
459           to set identifier to previously allocated Command Payload.  It
460           is used to set identifier for command when resending Command
461           Payload.
462
463         * Added silc_command_payload_encode_payload to encode Command
464           Payload buffer from SilcCommandPayload structure.
465
466         * Added silc_argument_payload_encode_payload to encode Argument
467           payload buffer from SilcArgumentPayload structure.
468
469 Wed Nov  8 21:03:28 EET 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
470
471         * Changed WHOIS command to support router connection on server side.
472           The whois request is always sent to router unless the server is
473           standalone server.  After server has received the reply from the
474           router will it send the reply to the client.
475
476         * Added silc_server_packet_broadcast into silcd/server.[ch] to
477           broadcast received broadcast packet.  The function is used only
478           by router.  The broadcast packet is always sent to the router's
479           primary route.
480
481         * Added silc_id_render function in lib/silcutil/silcutil.[ch] to
482           render given ID to printable string, for log files for example.
483
484 Tue Nov  7 22:14:19 EET 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
485
486         * Made basic router to router connections working.  At least they
487           can now connect to each other but nothing really works the way
488           they are supposed - yet.
489
490         * Added new initiator token to RouterConnection configuration
491           file in silcd/serverconfig.[ch].  It is used to tell whether we
492           are the initiator to the remote router or whether we'll expect
493           the other end to connect.
494
495         * Moved registering of listener task to silc_server_init, hence
496           the server starts listenning as soon as it is run, even if it
497           does not have connections to other routers.  Let's see how well
498           this will work.
499
500         * Changed default connection retry timeouts for more suitable in
501           silcd/server.h.
502
503         * Removed cipher and such arguments from silc_idlist_add_client
504           and silc_idlist_add_server prototypes from silcd/idlist.[ch].
505           Added new function silc_idlist_add_data to add the keys and stuff
506           to any ID entry.
507
508         * Added SilcIDListData structure and added it to SilcClientEntry
509           and SilcServerEntry as their first field in the structure.  This
510           way we can explicitly cast the ID entries to the SilcIDListData
511           structure and get common data for the entries.  In past, we had
512           to first check what type of connection it is and then cast it to
513           correct ID entry type.  Now, we can directly cast the opaque
514           pointer to the SilcIDListData (no matter what ID entry it actually
515           is) and get the data needed.
516
517 Mon Nov  6 21:56:12 EET 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
518
519         * Wow, found a bug in scheduler.  The scheduler uninitialized itself
520           in some circumstances even if threre were timeout tasks, though not
521           IO tasks, but tasks anyway.  Now fixed.
522
523         * Defined SilcServerConnection structure to hold connection specific
524           stuff about directly connected servers and routers.  The definition
525           is currently in silcd/server_internal.h.  I thought about having
526           a bit more important role fro this struct but for now it is used
527           only when connecting to other server (or router actually).
528
529         * Added connecting retry support in server when connecting to
530           router(s).  The retry feature implement exponential backoff
531           algorithm.  Also, added SilcServerParams structure to hold default
532           parameters for server.  For now, it include these retry settings
533           and are hard coded.  After server is moded to be as Silc Server
534           Library this structure will be more important.
535
536 Sun Nov  5 22:28:44 EET 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
537
538         * Changed client librarys channel->clients table to SilcList and
539           changed code accordingly.
540
541 Thu Nov  2 16:28:01 EET 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
542
543         * Changed client's channel table to SilcList and changed code 
544           accordingly.  Also changed SilcChannelClientEntry to include back-
545           pointer to the channel so that client entry can use that structure
546           as list as well and we have fast cross-reference to the channel.
547           This change dramatically decreased the complexity of channel
548           handling with client entry and vice versa (removed one extra
549           loop when searching for channel entry from many functions).
550
551         * Changed server->sim from table to SilcDList and changed code
552           accordingly.
553
554         * NAMES command can now be used from user interface.  It will show
555           the user list on the channel, neatly.
556
557         * Added realname pointer to SilcClientEntry in lib/silcclient/idlist.h.
558           Code now saves realname of the user if it becomes available.
559
560         * Renamed configure.in to configure.in.pre and made ./prepare
561           script to automatically add correct version string to
562           configure.in which it creates from configure.in.pre.
563
564 Wed Nov  1 17:21:26 EET 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
565
566         * NAMES command reply now shows users mode with the nickname when
567           joining to channel.
568
569         * Moved silc_client_ch[u]mode[_char] functions from 
570           silc/clientutil.[ch] to lib/silcclient/client.[ch].  Though, that
571           place sucks, they are utility functions and should be in some
572           other file.
573
574         * Fixed some unsigned int's to unsigned short's.  Patch by cras.
575
576         * Fixed contrib/getopt*.[ch] to not require config.h.  Patch by
577           cras.
578
579 Tue Oct 31 20:10:37 EET 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
580
581         * Updated README.
582
583         * Added TRQ (efficient deque and list library) into lib/trq.  This is
584           a very good list library that is currently used in the SILC.  Defined
585           SilcList API over the library because I didn't like the API very
586           much.  See lib/trq/silclist.h for the API and examples of how to
587           use the API.  Fixed various places in the code to use the new
588           SilcList API. The SilcList is meant for lists that has a structure
589           already defined as a list.  It is not suitable to add just some
590           context to the list (in TRQ, the context is the list actually).
591
592           So, I defined SilcDList that can be used for the purpose where 
593           predefined list structure does not exit.  This can be used as
594           such list.  Now some context just can be added to the SilcDList.
595           Currently this list is not used in the SILC just yet, though there
596           are a lot places where this can replace dynamically allocated
597           tables and I will fix these places, later, to use SilcDList.
598           See lib/trq/silcdlist.h for SilcDList (they are all inline functions,
599           and use TRQ internally).
600
601           Also fixed some annoying warning messages that the original TRQ
602           code generated.  Also minor changes to TRQ's Makefile.in.
603
604         * Added support for querying by Client ID to both WHOIS and 
605           IDENTIFY commands into server, as required by the protocol.
606
607         * Removed method function pointers from SilcBuffer structure. They
608           weren't used to anything and just increased the context size for
609           no good reason.  This change also made silc_buffer_alloc and
610           silc_buffer_free functions inline functions.
611
612         * Disabled command flooding detection support until it's fixed so 
613           that it accepts commands in but does not execute them more than once
614           in two seconds.
615
616         * Added silc_net_localhost(), to return local hostname, into
617           lib/silcutil/silcnet.[ch].  Also added client->hostname pointer
618           that must be initialized before calling silc_client_init.
619
620         * Added new function: silc_server_send_notify_on_channels to send
621           notify messages to all channels client has joined.  It is assured
622           that the message is sent only once per client.
623
624         * Moved silc_log_format (from lib/silcutil/silclog.[ch] into
625           lib/silcutil/silcutil.[ch] as silc_format function.  The new 
626           function is generic and is used by server as well, not only by
627           the logging routines.
628
629         * Added new SKE status type: SILC_SKE_STATUS_BAD_VERSION to indicate
630           the provided version string was not acceptable.  Added new function:
631           silc_ske_check_version into lib/silcske/silcske.h.  The function
632           must be implemented by the application (client or server) and it
633           does not reside in the SKE library.  The function checks the version
634           string remote end sent.
635
636         * Added back pointers (to opaque context and to SilcSocketConnection) 
637           into SilcPacketContext structure into lib/silccore/silcpacket.h.
638
639         * Added silc_packet_context_dup into lib/silccore/silcpacket.[ch] to
640           duplicate packet context structure.
641
642         * Changed `notify' client operation to send same arguments as client
643           receives from server except for ID's.  ID's are mapped to correct
644           ID entry and that is returned.  Also, if channel entry is not sent
645           by server but the notify is for channel the channel entry is sent
646           to application (otherwise application doesn't know that it is for
647           channel (library gets it from packet's Destination ID)).
648
649         * Added silc_client_remove_from_channels into client library to 
650           remove a client from all channels it has joined to.  Used when 
651           received SIGNOFF notify from server.  Added also new function
652           silc_client_replace_from_channels to replace old ID entry with
653           new ID entry on all channels.  Used when received NICK_CHANGE
654           notify from server.
655
656         * Fixed ID Cache list handling in silc_idlist_get_client in 
657           lib/silcclient/idlist.c.  Also, added silc_idlist_get_client_by_id
658           to get (or query) client by ID.
659
660         * Updated TODO list.
661
662         * Added connection authentication status message defined by the
663           protocol: SILC_CONN_AUTH_OK and SILC_CONN_AUTH_FAILED and added the
664           support for these into the code in client and server side.
665
666         * Added generic function silc_client_send_command to send any command
667           with variable argument list.  Application should use this function
668           to send commands if the command functions provided by the library
669           does not suite for the application's user interface needs.
670
671         * Added new `failure' client operation.  Application is notified about
672           received failure packet if client is executing a protocol.  In this
673           case the protocol's execution has failed.
674
675         * Added SKE's end notify to send the SKE_SUCCESS notify message that
676           is required by the protocol.
677
678         * Added SILC_PROTOCOL_STATE_FAILURE to indicate received failure
679           packet from remote.  SILC_PROTOCOL_STATE_ERROR indicates local
680           error at our end.
681
682         * Added status flag to SilcSKE object to indicate realtime status
683           of the SKE protocol.
684
685         * Application receives now exactly same command reply arguments as
686           the library receives from server.  However, if ID is received the
687           corresponding ID entry is returned to the application (eg. Client
688           ID is mapped to correct SilcClientEntry entry and that is returned).
689           Changed command_reply client operation due to this change.
690
691         * Changed all ID's in commands and in command replys as ID Payloads.
692           Change affected both client and server side codes.
693
694           All ID's sent in SILC network (with execption of ID's in SILC
695           Packet header) are sent in ID Payload to support variable length
696           ID's.
697
698         * Server now notifies nick changes and notifies all clients on
699           the channels about the new nickname (about the new Client ID,
700           actually).
701
702         * Implemented CMODE command to change channel modes. Supports all
703           channel modes defined by the protocol specs except ban and invite
704           lists. (Also, private channel key mode is supported but support for
705           setting private channel key in client is missing, thus, this mode
706           has no effect on client side (except that server requires that the
707           client uses private channel key and normal channel traffic does not
708           work anymore)).
709
710           Also, invite mode works per se, but INVITE command does not work
711           yet correctly, so you can set channel as invite only channel but
712           inviting clients to the channel does not work (it is yet to be
713           thought what's the best way to do it).
714
715         * Added new command SILC_COMMAND_CUMODE to change user mode on the
716           channel.  Defined user modes: CHANNEL_FOUNDER and CHANNEL_OPERATOR.
717           Implemented CUMODE command to change user's mode on the channel.
718           Supports all modes defined by the protocol specs.
719
720         * Added NAMES command reply to return users modes on the channel.
721
722         * Removed unnecessary and slow ciphers from lib/silccrypt.
723
724         * Set SO_KEEPALIVE option to connection sockets by default.
725
726         * Added new command reply status: SILC_STATUS_USER_NOT_ON_CHANNEL.
727
728         * Added notify types: MOTD, CMODE_CHANGE and CUMODE_CHANGE.  Also,
729           redefined the Notify Payload into protocol specs.
730
731         * Added silc_id_payload_parse_id to get ID directly from raw
732           ID payload data.
733
734 Mon Oct  9 20:57:02 EEST 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
735
736         * Changed SILC_COMMAND_IDENTIFY in protocol specification to 
737           accept searching by Client ID as well.
738
739         * Added support for LEAVE and SIGNOFF notify types in client library.
740
741         * Added silc_id_payload_parse_data into lib/silccore/silcpayload.[ch]
742           to parse ID Payload from raw data.
743
744 Sun Oct  8 19:33:08 EEST 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
745
746         * Added flags parameter into silc_ske_assemble_security_properties
747           function in lib/silcske/silcske.[ch].
748
749         * Changed notify client operation to fit better for notify messages
750           sent by server.  The notify payload received from server is now
751           passed to the application (after parsing it to SilcNotifyPayload).
752           It is application's responsibility to retrieve the arguments
753           from the payload and show the message the way it wants.  The message
754           sent by server is implementation specific.
755
756         * Changed public keys to comply with the protocol specification.
757           Old public keys are not supported anymore and are not compatible.
758
759         * Removed nickname from Channel Payload as the latest draft removed
760           it.  The client must resolve the nickname from the NAMES command
761           reply received when it joined the channel.
762
763           Also, changed all channel_xxxx_payload to channel_payload_xxxx.
764
765 Sat Oct  7 21:55:01 EEST 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
766
767         * Fixed some errors in protocol specification drafts.
768
769         * Created lib/silccore/silcnotify.c to implement Notify Payload
770           encoding and decoding, lib/silccore/silcpayload.[ch] to implement
771           generic payloads described by protocol specifications.  The file
772           includes implementations for ID Payload and Argument Payload.
773
774         * Changed Command Payload implementation to use the new Argument
775           Payload.  Changed command_xxxx_payload to command_payload_xxxx
776           to comply with SILC coding conventions.
777
778         * Added suppport for Argument Payload handling in Notify Payload
779           implementation as protocol requires it.  Added the new support
780           into server and client lib as well.
781
782 Thu Oct  5 21:16:28 EEST 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
783
784         * Added support for multiple nicknames on same channel.  [n] is
785           added locally to the nickname if there are more than one same
786           nicknames on the channel.
787
788         * Server now sends all nicknames that matched WHOIS request.
789           Client also shows the list received from server.
790
791         * Added TOPIC command to client side.  User can now set and show
792           current topic on channel.
793
794         * Added MOTD command to client and server.  Also, server sends the
795           motd when client connects to the server.
796
797         * Changed version strings to comply ISO 8601.
798
799 Wed Oct  4 23:29:06 EEST 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
800
801         * Fixed protocol error handling in client library.  It should now
802           cope even if the SKE fails for some reason.
803
804         * Made new protocol specification drafts for submitting to IETF.
805
806         * Implemented TOPIC command to server in silcd/command.c.
807
808         * Added two new notify types into lib/silccore/silcnotify.h:
809           SILC_NOTIFY_TYPE_NICK_CHANGE and SILC_NOTIFY_TYPE_TOPIC_SET to
810           notify nickname change and topic setting/change on a channel.
811
812         * API change of command_reply operation in client library.  The
813           application gets now the status type received from server as well.
814
815 Sat Sep 30 16:57:42 EEST 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
816
817         * Removed the function just added to lib/silcutil/silcschedule.[ch].
818
819         * Cras fixed and optimized the packet handling even further and
820           it should work now.  Minor change to the prototype of function
821           silc_packet_receive_process in lib/silccore/silcpacket.[ch].
822
823 Sat Sep 30 08:48:47 EEST 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
824
825         * Added new function into lib/silcutil/silcschedule.[ch]:
826           silc_schedule_with_fd to select() a specified fd.  The function
827           returns after timeout expires or data arrives or goes.  The
828           function is used by packet routines to wait that all data is
829           received from network.
830
831         * Fixed data reading from network in lib/silccore/silcpacket.c.
832           The code now assures that all data is read from the fd and then
833           continues packet processing.  This was a bug fix since the code
834           used to drop some data in some circumstances.
835
836         * Added new function into lib/silcclient/client.[ch]:
837           silc_client_start_key_exchange to start key exchange after
838           connection has been established to server.  The code internally
839           now uses this funtion but its main purpose was to provide it
840           for applications that perform their own connecting.  After
841           application has created a connection it merely calls this
842           function to start the key exchange between client and server.
843           The library takes care of everything else after that.
844
845           Updated also lib/silcclient/README to explain the usage of
846           this new function.
847
848         * Do not send to application information that connection has
849           been established.  Application gets notified it by connect
850           operation anyway.
851
852 Thu Sep 28 23:40:19 EEST 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
853
854         * Applied cras's patch to add silc_schedule_one function.  The
855           function runs scheduler once and returns.
856
857         * Fixed the scheduler after cras messed it up.  The timeout
858           handling works now as it's supposed to work.
859
860         * Added into lib/silccore/ silcnotify.h to include notify
861           message types support.  Changed silc_server_send_notify*
862           functions, in server.[ch], to support those new notify types.
863           Added the support for the notify types into client library,
864           as well.  Added new notify client operation into ops.h in
865           lib/silcclient/.
866
867         * Changed silc_server_packet_send_to_channel to send normal
868           packets instead of just channel message packets.  The function
869           is now used to send the notify packets to channels.  It is not
870           used to send channel message packets anymore, as server never
871           sends them anymore.
872
873         * Added explicit casting into lib/silcutil/silcbuffmt.c to few
874           va_arg()s as it seems to require it nowadays.  I guess, if SILC
875           is compiled with older va_arg() the new code should work anyway.
876
877 Wed Sep 13 18:10:14 EEST 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
878
879         * Splitted core library.  Core library (lib/silccore) includes
880           now only SILC protocol specific core (and common) components.
881           Created new utility library (lib/silcutil) that includes more
882           generic purpose stuff.  The stuff for util library was taken
883           from the old core library.  This was minor and easy split.
884
885         * Created SILC Client Library (lib/silcclient) that includes
886           implementation of the SILC client without user interface.  This
887           was major move from silc/ directory.  The code has been changed
888           so that it is transparent towards the user interface.  The
889           silc/ directory includes now the same user interface as before
890           and it uses the new client library.  Read lib/silcclient/README.
891           Basicly, the client library performs everything else related
892           to SILC except user interface handling.  Also, configuration
893           files are considered to be part of user interface and library
894           does not handle them.
895
896           This change also changed a lot of structures, function naming etc.
897           Most important change was that SilcClientWindow object was
898           renamed to SilcClientConnection in the client library.  Created
899           also new file lib/silcclient/ops.h.  Also added new files
900           silc/local_command.[ch] and silc/client_ops.[ch].
901
902           All these changes were made to make it easier for user interface
903           designers to create what ever user interface for the SILC client
904           they want.
905
906           It is also expected that the server will be moved to lib
907           directory as well and SILC Server Library will be created;
908           sometimes in the future.
909
910         * Removed Local commands from lib/silccore/silccommand.h as
911           they are application specific and new client library does not
912           handle any of those anymore.
913
914         * Several functions moved to lib/silcutil/silcutilc.[ch] from
915           old client implementation in silc/.
916
917         * Added support for callback functions in SILC_LOG_* macros.
918           Application can now set its own callbacks that will be called
919           instead of using the default functions that will always print
920           the debug messages to stderr (or stdout).  Also, debugging can
921           now be disabled by setting silc_debug to FALSE and re-enabled by
922           setting it to TRUE.  Note, that logging will still work even
923           if debugging is disabled.
924
925           New functions in lib/silcutil/silclog.[ch]: silc_log_set_callbacks,
926           silc_log_reset_callbacks, silc_log_set_debug_callbacks and
927           silc_log_reset_debug_callbacks.
928
929         * To enable debugging in silc client one must give now -d
930           option on command line.
931
932         * Changed silc_schedule_init to automatically allocate task queues
933           if they are not allocated before calling it.
934
935 Thu Sep  7 10:49:33 EEST 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
936
937         * Added GMP 3.1 into math library.
938
939 Sun Aug 20 21:27:26 EEST 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
940
941         * Added SILC_PACKET_REMOVE_CHANNEL_USER to remove a client from
942           a channel in SILC network.  The packet is used by servers and
943           routers to notify other routers that user has left a channel.
944           This little feature was missing until now.  Added the feature
945           to protocol specification as well.
946
947           Added functions: silc_server_send_remove_channel_user and
948           silc_server_remove_channel_user into server.[ch].
949
950         * Added SILC_PACKET_REKEY and SILC_PACKET_REKEY_DONE into
951           lib/silccore/silcpacket.h.  However, they are not implemented
952           yet.
953
954 Sat Aug 19 23:04:16 EEST 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
955
956         * Fixed joining to a channel and sending channel messages
957           between server and router.  The channel message sending should
958           now work inside a cell.
959
960 Tue Jul 25 20:46:13 EEST 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
961
962         * Fixed the private message sending between server and router.
963           The private message sending should now work inside a cell.
964
965         * Added silc_server_replace_id into server.[ch] to replace
966           existing ID in the SILC network.
967
968         * Added silc_idlist_find_server_by, silc_idlist_replace_client_id
969           and silc_idlist_replace_server_id into idlist.[ch] in server.
970
971 Mon Jul 24 18:33:31 EEST 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
972
973         * Fixed the server to server connections.  Server can again now
974           connect to router.  Router to router connections probably does
975           not work just yet.
976
977 Thu Jul 20 13:15:01 EEST 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
978
979         * Added dynamic protocol registering support.  Now protocols can
980           registered and unregistered on the fly.  Patch by cras.
981
982 Wed Jul 19 19:08:46 EEST 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
983
984         * Added lib/contrib directory to hold routines that some platforms
985           don't have but are needed by SILC.
986
987         * Added getopt.c, getopt1.c and getopt.h from GNU C library
988           into lin/contrib to provide getopt() and getopt_long() for
989           those who don't have it.
990
991 Tue Jul 18 20:41:20 EEST 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
992
993         * Added AWAY command to client.  When away message is set and
994           client receives a private message packet the client automatically
995           replies to the sender with the away message.
996
997         * Fixed a bug in lib/silcmath/mpbin.c: silc_mp_mp2bin.  This
998           bug seemed to be the cause of recent problems when compiling
999           with gcc-2.95.
1000
1001         * Added version detection support to SKE protocol specification
1002           and added the new changes to the SKE implementation as well.
1003           There were other minor changes in the SKE protocol as well.
1004
1005           Many changes in lib/silcske/silcske.[ch] and in
1006           lib/silcske/payload.[ch].
1007
1008         * Added ^U functionality, clear input line.  Patch from cras.
1009
1010 Mon Jul 17 23:33:26 EEST 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
1011
1012         * Mainly small bugfixes on core library.  Fixed some debugging
1013           logging and buffer overflow in silclog.c.
1014
1015         * Updated config.sub and config.guess on the distribution tree.
1016
1017 Sat Jul 15 15:33:48 EEST 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
1018
1019         * Added command lagging support in server. Client may execute
1020           commands now only once in two seconds.
1021
1022 Thu Jul 13 22:10:21 EEST 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
1023
1024         * Optimized packet reception. MAC computation and checking is now
1025           also more optimized.  A lot previously duplicated code is now
1026           used as generic by both client and server.
1027
1028         * Fixed key pair generation in clientutil.c
1029
1030 Wed Jul 12 18:28:07 EEST 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
1031
1032         * Added into lib/silccore/silcbufutil.[ch] new function;
1033           silc_buffer_realloc.
1034
1035         * Moved generic packet sending/encryption functions to 
1036           lib/silccore/silcpacket.[ch] from client and server.  Some
1037           rewriting of the functions.
1038
1039         * Moved all generic packet reception/decryption functions to
1040           lib/silccore/silcpacket.[ch] from client and server.  The
1041           packet processing is now much cleaner in both client and server.
1042           These were major changes in both client and server.
1043
1044         * Created many common functions in server to do packet sending.
1045           Previously code were duplicated a lot, this has been removed
1046           with these changes.
1047
1048 Tue Jul 11 20:27:26 EEST 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
1049
1050         * Rewrote major parts of the ID cache system.  Don't know 
1051           whether it is better now or not but at least the API is more
1052           cleaner now.
1053
1054         * Major rewrite on ID cache stuff on client because of the ID
1055           cache API changes.  Added idlist.c to client.
1056
1057         * Also major rewrite on ID cache stuff on server as well.
1058           Major rewrite of idlist.[ch]. SilcXXXList's are now named
1059           SilcXXXEntry's.  We won't keep anymore idlist specific pointers
1060           in hand, instead they are all put into the ID cache system now.
1061           All server_idlist_* routines uses ID cache now instead of
1062           traversing its own lists (those lists does not exist anymore).
1063           SilcIDList though still exists.  Also, SilcXXXEntry's are
1064           now pointers.
1065
1066 Sun Jul  9 15:19:24 EEST 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
1067
1068         * Finally made the SKE implementation compliant to the protocol
1069           specification.  All mp integers are now binary encoded as
1070           opposed being HEX encoded.
1071
1072         * Added lib/silcmath/mpbin.[ch].  Encoding mp intergers to and
1073           from binary data.
1074
1075         * Added into lib/silccore/silcutil.[ch] PEM encoding/decoding
1076           functions: silc_[encode/decode]_pem.  Also added function
1077           silc_encode_pem_file to PEM encode with newlines ('\n') for
1078           saving into a file.
1079
1080         * SILC public keys are now encoded either PEM or binary.  Same
1081           option is for private keys as well.  By default private keys
1082           are binary encoded and public keys PEM encoded.  Silly HEX
1083           encoding were removed.
1084
1085         * Added into lib/silccrypt/silchash.[ch] silc_hash_fingerprint
1086           function to create fingerprints.
1087
1088         * Fixed a bug in SHA1; does not change the original data anymore.
1089
1090         * Partly implemented INFO command on client and server side.
1091           Fixed CLEAR command.  Changes to SERVER command; show current
1092           server(s) when giving command without arguments.  Added
1093           VERSION command to client.
1094
1095         * Added check to server that unregistered connections cannot
1096           execute commands (unless it is specificly allowed).
1097
1098 Thu Jul  6 18:12:24 EEST 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
1099
1100         * Fixed screen refresh.
1101
1102         * Fixed channel joining bug from client.  On some circumstances
1103           client tried to join to a channel it had already joined.
1104
1105         * Added public key verification process into client's protocol.c.
1106           The client now verifies the public key from user and saves
1107           it into ~./silc/serverkeys/ directory. 
1108
1109           Added into: clientutil.[ch]: silc_client_verify_server_key.
1110
1111         * Changed SKE protocol's silc_ske_initiator_finish function
1112           to accept callback function that verifies the received public
1113           key.  Removed old silc_ske_verify_public_key function.
1114
1115 Wed Jul  5 19:19:02 EEST 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
1116
1117         * Added into silcpkcs[ch]: silc_pkcs_public_key[_data]_set and
1118           silc_pkcs_private_key[_data]_set.
1119
1120         * Made the password and public authentication more cleaner in
1121           server's protocol.c.
1122
1123         * Removed historic and obsolete protocol `channel_auth' from
1124           both client and server.
1125
1126         * Removed wrong way of sending command status messages from
1127           server to client in server's command.c.  The old way violated
1128           protocol specification.  
1129
1130           Changes to silccore/silccommand.[ch]: removed
1131           silc_command_encode_status_payload -> not needed anymore,
1132           changed silc_command_encode_payload_va to accept extra
1133           argument on variable argument list.  The argument type must
1134           now be provided to the function.  Also, added new function:
1135           silc_command_encode_reply_payload_va which is same as
1136           normal command_encode_payload_va except command status type
1137           is provided as extra argument.
1138
1139 Tue Jul  4 18:26:39 EEST 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
1140
1141         * Added ~./silc directory handling.  The directory includes the
1142           public and private keys for the client.
1143
1144           Added silc_client_check_silc_dir, silc_client_create_identifier
1145           and silc_client_load_keys.
1146
1147         * Implemented SILC protocol compliant public key.  Added public
1148           and private key saving to and loading from files.
1149
1150           Added into silcpkcs.[ch]: silc_pkcs_encode_identifier,
1151           silc_pkcs_public_key_encode[_data], silc_pkcs_public_key_decode,
1152           silc_pkcs_private_key_encode[_data], silc_pkcs_private_key_decode,
1153           silc_pkcs_public_key_alloc, silc_pkcs_public_key_free,
1154           silc_pkcs_private_key_alloc and silc_pkcs_private_key_free.
1155
1156           Implemented: silc_pkcs_save_[public/private]_key[_data] and
1157           silc_pkcs_load_[public/private]_key.
1158
1159 Mon Jul  3 18:51:27 EEST 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
1160
1161         * Added silc_server_get_route (route.[ch]) to get connection
1162           data for the fastest route for given ID.
1163
1164         * Implemented INVITE command on client and server.  The command
1165           were re-defined in the SILC Protocol Specification and the
1166           implementation now complies with the specification.
1167
1168         * Implemented PING command on client and server.
1169
1170         * Implemented NAMES command on client and server.  The server side
1171           supports currently only normal server not router server yet.
1172           Some changes to NAMES definition in SILC protocol specification.
1173
1174 Sun Jul  2 18:23:01 EEST 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
1175
1176         * Implemented LEAVE command on client and server.
1177
1178         * Previously deprecated SILC_PACKET_FORWARDED flag is now in use 
1179           again.  This change was made to the protocol as well.  Server
1180           should not violate the protocol specification anymore.
1181
1182 Fri Jun 30 14:03:26 EEST 2000  Pekka Riikonen <priikone@poseidon.pspt.fi>
1183
1184         * Added SOCKS4 and SOCKS5 support to SILC client.  SOCKS5
1185           was tested.  SOCKS4 was not but should work anyway.