[Olsr-dev] New plugin: telnet

[Ferry Huberts]

On 20/02/13 09:54, equinox wrote:
> Hi,
>
> Am 2013-02-20 07:59, schrieb Henning Rogge:
>> On 02/20/2013 03:56 AM, equinox wrote:
>>> Hi,
>>>
>>> I wrote a small telnet plugin which allows to change some configuration
>>> options on the fly (without the need to restart the daemon). See detailed
>>> description below...
>>> I also added a tar.gz containing the patch set which is based on current
>>> master branch of the official git repo.
>>>
>>> Please have a look and tell me if you are willing to accept it. Also
>>> it would
>>> be nice if some of you could check if everything the plugin does is
>>> ok. I tested
>>> with valgrind and don't think that there are memory leaks but it would
>>> be good if
>>> somebody who knows the internal data structures would take a look at
>>> what i'm doing.
>>> Especially with adding/enabling and removing/disabling interfaces i'm
>>> not complete
>>> certain....
>>
>> Any relationship to the telnet code I had on the former master-branch
>> some time ago?
>
> hmmm... no i didn't knew that you where working on that ... at least not
> for this version of olsrd. I saw that you have a telnet server at the
> oonf api and thought this would be nice to have for olsrd (v1) as well.
> The code i added contains a little bit of jsoninfo (initialization of
> server socket) but besides that it's brand new.
>
> regards
>   christian
>
>
>


I'm not at all convinced that adding such a plugin is a good idea.
It might be but your mileage may vary: in general olsrd v1 is UNSUITED 
for dynamic configuration.
Some stuff might work, other stuff might not work.

Also, it seem like we already have something of a telnet server, and 
it's called txtinfo.

The amount of code duplication already is well above tolerance levels 
and people duplicating functionality is not doing anything good to it.

I'm not particularly open to adding any more plugins to the tree until 
people make clear why the plugin must be added and why the functionality 
can't be added to any of the plugins already present in the tree.


-- 
Ferry Huberts