ipedrazas / drone-helm

Helm (Kubernetes) plugin for drone.io
MIT License
121 stars 90 forks source link

uppercase prefixed secrets during replacement #47

Closed danielkrainas closed 7 years ago

danielkrainas commented 7 years ago

drone exposes secrets as uppercase keys. If your prefix was lowercase (e.g: dev) it would cause the prefixed form of the variables not to be found (e.g: dev_API_SERVER) My change uppercases the prefixed form (e.g: DEV_API_SERVER) as expected. It also breaks up debug() into debugEnv() and debug() as it was useful to have the environment printed out before any processing was attempted to determine the bug.