[OpenSIPS-Devel] New MI module names

[Bogdan-Andrei Iancu]

I second that, it is a bad and useless practice to include the name of 
the module in the modparam names - these params are to be exclusively 
used in combination with the name of the module.

Regards,

Bogdan-Andrei Iancu

OpenSIPS Founder and Developer
   https://www.opensips-solutions.com
OpenSIPS Summit 2019
   https://www.opensips.org/events/Summit-2019Amsterdam/

On 03/23/2019 01:32 PM, Dan Pascu wrote:
> One thing that came to mind to help mitigate the need for changing everything when a module is renamed and also simplify the naming for module parameters is to remove the redundant module names prefixing each module parameter name.
>
> Right now they are like:
>
> loadmodule "mi_http"
> modparam("mi_http", "mi_http_root", "some_path")
>
> we can get rid of the module name prefix, since module parameters are already in the namespace of the module, making the prefix redundant. so it will be like:
>
> loadmodule "mi_http"
> modparam("mi_http", "root", "some_path")
>
>
>> On 22 Mar 2019, at 14:33, Vlad Patrascu wrote:
>>
>>> Hi Dan,
>>>
>>> Indeed, those module parameters will be renamed accordingly, it's still on the todo list.
>>>
>>> As for the module names, they should now reflect the transport protocol and not the encoding, since the majority of modules use JSON. 'mi_html' and 'mi_xmlrpc_ng' are the only exceptions. Even though they are also over HTTP, it makes sense to differentiate them by encoding since they don't align with the internal MI interface(which is now JSON-RPC) and do a translation.
>>>
>>> Regards,
>>>
>>> Vlad Patrascu
>>> OpenSIPS Developer
>>> http://www.opensips-solutions.com
> --
> Dan
>
>
>
>
>
> _______________________________________________
> Devel mailing list
> Devel at lists.opensips.org
> http://lists.opensips.org/cgi-bin/mailman/listinfo/devel