GitHub user tenthe added a comment to the discussion: Removal of Temporary Migration Step in nginx `Dockerfile`
Thanks! I believe a modified version of option 2 from your list is the best approach. Since this primarily affects users who utilize `HttpServerProtocol`, we could ship the image with a `default.conf` that **does not** include the endpoint configuration. Given that users must manually expose the port to the service anyway, they should also be able to provide their own `default.conf` with the necessary endpoint definition. To make this as seamless as possible for those using `HttpServerProtocol`, I suggest adding a **detailed explanation** in the element documentation, along with a dedicated section in the main documentation. We could also include a ready-to-use configuration template that users can copy and paste. This way, users who don't need the adapter get a simple, clean setup by default, while those who do can easily configure it. Since they already know the service name, there's no need for an additional environment variable. @obermeier, what do you think? GitHub link: https://github.com/apache/streampipes/discussions/3446#discussioncomment-12175520 ---- This is an automatically sent email for [email protected]. To unsubscribe, please send an email to: [email protected]
