eclipse-velocitas / vehicle-app-python-sdk

vehicle-app-python-sdk
Apache License 2.0
7 stars 18 forks source link

[Improvement]: Rename native middleware variable 'SDV_MQTT_ADDRESS' #90

Open BjoernAtBosch opened 1 year ago

BjoernAtBosch commented 1 year ago

Description

The name is a bit misleading: "MQTT" is a protocol and cannot have an address. The address refers to the message broker.

Suggested Solution

Replace name of the variable by "SDV_MQTTBROKER_ADDRESS" (or maybe "SDV_MQTT_BROKER_ADDRESS" (depending on the possible service names).

Maybe keep old name for a while with deprecation warning if still used.

Alternatives

No response

Additional Context

Needs to be fixed in Python and C++ SDK plus devenv-runtimes.