Closed yanzv closed 2 years ago
Cool. I'll look at it soon.
It looks OK to me. Thanks for dealing with the code duplication. To run on separate networks again, one would need to revert the changes to application.conf, I guess. Do you all there prefer an empty String to an Option[String] of None?
Great. To run on a separate network you would just configure authentication via ENV variables.
username: ${?REST_PRODUCER_USERNAME}
password: ${?REST_PRODUCER_PASSWORD}
I think Option[String] would be preferable but I didn't want to change the method signature.
Were these values unnecessary or wrong then? (I'm making notes in case it does need to go back one day.)
sasl.jaas.config: "org.apache.kafka.common.security.plain.PlainLoginModule required username=\""${KAFKA_USERNAME}"\" password=\""${KAFKA_PASSWORD}"\";"
security.protocol: SASL_SSL
This is when you are connecting to SASL_SSL authenticated kafka endpoint otherwise the connection will fail if the protocol is PLAINTEXT
You might be interested in webapp/docker.sbt. Recently we've been having sbt build the docker images. It works in two stages (as your system does) and probably gets to skip some of the time-consuming intermediate steps. It has seemed fast to me.
For DART in the box eidos is going to be running within the same docker network as the rest of the applications and I would like to be able to configure for wmexchanger to connect to Kafka and REST endpoints directly without authentication. @kwalcock please let me know if you have any concerns with these config changes. Also, squash merge to if you are going to accept the pr.
Thanks!