Yes, this is exactly how other plugins handle the situation. The :8000
part is just hard-coded in dev settings.
I understand that you cannot do this here in your plugin. So instead I would suggest that you pass the full LMS url to your superset container via an environment variable. This variable will be different in production and development.
Pretty close It’s actually “local-docker-compose-dev-services”. I can understand why this would not be a very intuitive name…
To sum it up, you need to add this value to the “local-docker-compose-services” production patch:
environment:
- LMS_FULL_URL = "{% if ENABLE_HTTPS %}https{% else %}http{% endif %}://{{ LMS_HOST }}"
And this one to the “local-docker-compose-dev-services” patch:
environment:
- LMS_FULL_URL = "http://{{ LMS_HOST }}:8000"