Allow to add custom start parameters to instances

[Anzic23]

same error - https://stackoverflow.com/questions/59898050/axios-ssl-error-with-node-12-ssl-routinesssl-choose-client-versionunsupporte
this script stopped working.
samsung-air.txt

Node.js v12.16.2
NPM 6.14.4
js-controller 3.0.20

2020-04-30 00:31:27.293 - error: javascript.0 (14338) uncaught exception: 140145197954880:error:1425F102:SSL routines:ssl_choose_client_version:unsupported protocol:../deps/openssl/openssl/ssl/statem/statem_lib.c:1942:
--
2020-04-30 00:31:27.293 - error: javascript.0 (14338) Error: 140145197954880:error:1425F102:SSL routines:ssl_choose_client_version:unsupported protocol:../deps/openssl/openssl/ssl/statem/statem_lib.c:1942:

[Apollon77]

Yes, we will add error handlers for all adapters that use it, but in fact ... what is wrong with the certifictaes?

[Apollon77]

Can you try to generate new certificates in iobroker? Use "iobroker cert" commands for that!

[Apollon77]

ANd: The full error/stack trace would be interesting. I would say you need to make sure to handle all errors in your script correctly. I do not nsee any topic for js-controller here ...

[Apollon77]

PS: or is this issues meaned to be a "feature request" to add "--tls-min-v1.0" as parameters when we execute certain adapter processes to allow insecure communication methods?

[Anzic23]

I'm not sure I know how to correctly install the certificate from the module instructions https://github.com/cicciovo/homebridge-samsung-airconditioner/blob/master/README.md

[Anzic23]

ANd: The full error/stack trace would be interesting. I would say you need to make sure to handle all errors in your script correctly. I do not nsee any topic for js-controller here ...

perhaps. But on previous versions it worked without problems. error appeared after update nodejs and js-controller

[Anzic23]

PS: or is this issues meaned to be a "feature request" to add "--tls-min-v1.0" as parameters when we execute certain adapter processes to allow insecure communication methods?

Yes, I think

[Anzic23]

any news?

[Apollon77]

any news?

Not implemented till now ... PRs welcome

[foxriver76]

Not really sure what is the goal here? new io-pack flag to allow starting adapters with --tls-min-v1.0?

[Apollon77]

No idea would be to make it generic and allow to specify "nodejs params to add", so in ioPackage have a common.nodejsProcessParams = '--tls-min-v1.0 --whateverparameelse'

So the adapte rdev can specify params that are used when starting the adapter

[foxriver76]

Yes, thus compact groups where one adapter would use a flag would alle be started with this flag of course. Will this may be a problem?

[Apollon77]

or the definition is that an adapter with thsi flag can never run as compact mode ... exactly because of this. I would not risk side effects here

[Apollon77]

We have another usecase now ... ioBroker/ioBroker.javascript#1022 would require to set --unhandled-rejections=warn as parameter to not restart on unhandled promises.

I propose something like

common.processCliParameters = ["first", "second", ...]

[foxriver76]

Ok, then we say controller will start the instance as own process even if added to a compact group? We can also get some awareness via the schema to avoid usage of compact + the new flag. However, controller still needs to handle rule breaks.

[Apollon77]

meeehh ... you are right ... Elevating the parameterrs for the comapct group process i smaybe no good idea ... interesting ... yes needs to be considered

[Apollon77]

Another usecase ioBroker/ioBroker.parser#68