[Olsr-dev] New plugin: telnet

[Henning Rogge]

Maybe it would be possible to combine both plugins... keep the current
txtinfo commands as telnet commands and have some more commands.

The other option would be to put the telnet server into the core and
let the plugins use it.

But you are right, the amount of things that can be changed during
runtime is limited with the current codebase.

Henning Rogge

On Wed, Feb 20, 2013 at 5:50 PM, Ferry Huberts <(spam-protected)> wrote:
>
>
> 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
>
>
> --
> Olsr-dev mailing list
> (spam-protected)
> https://lists.olsr.org/mailman/listinfo/olsr-dev



-- 
We began as wanderers, and we are wanderers still. We have lingured
long enough on the shores of the cosmic ocean. We are ready at last to
set sail for the stars - Carl Sagan