cf-toolsuite / home

The home for ideas and exploration. Mainly documentation, videos and discussion.
Apache License 2.0
0 stars 1 forks source link

Docker config .env samples contain questionable looking environment variables #9

Open sneal opened 5 months ago

sneal commented 5 months ago

While Java will let you use env vars with dashes like PIVNET_API-TOKEN, most shells will not. We should use PIVNET_API_TOKEN

pacphi commented 5 months ago

Spring config convention. Camel-case properties are translated and recognized accordingly. See with relaxed binding an "_" denotes a nested property, a "-" denotes a change in case in a variable name. Also see https://docs.spring.io/spring-boot/docs/current/reference/html/features.html#features.external-config.typesafe-configuration-properties.relaxed-binding.environment-variables.

Maybe a way we could go is just remove "-" from environment variable names?