Change in ...libosmocore[master]: vty/vty.c: log received commands from telnet interface
gerrit-no-reply at lists.osmocom.org
Tue Jul 30 16:57:53 UTC 2019
neels has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmocore/+/14974 )
Change subject: vty/vty.c: log received commands from telnet interface
Patch Set 3:
I think this is a good idea, but causes unit tests to get unexpected log output.
Also it changes the application's user interaction behavior just by using a different libosmocore version, which we should avoid.
I think you need to solve this by adding a flag that keeps VTY command logging disabled by default, so that all the unit tests that work with both VTY and log output remain unchanged and keep passing.
This could either be a boolean flag that the main() scope of each application sets to true in C,
and/or this could be set to true by an actual vty config item.
An example for such a flag would be osmo_fsm_term_safely(), e.g. set to true in msc_main.c, but off by default to not cause inadvertent behavioral changes from a new libosmocore version.
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/14974
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Owner: Vadim Yanitskiy <axilirator at gmail.com>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Vadim Yanitskiy <axilirator at gmail.com>
Gerrit-Reviewer: pespin <pespin at sysmocom.de>
Gerrit-CC: neels <nhofmeyr at sysmocom.de>
Gerrit-Comment-Date: Tue, 30 Jul 2019 16:57:53 +0000
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the gerrit-log