tellerops / teller

Cloud native secrets management for developers - never leave your command line for secrets.
https://github.com/tellerops/teller
Apache License 2.0
2.83k stars 186 forks source link

Add quotes around values that include whitespace characters #241

Closed zshehov closed 4 months ago

zshehov commented 6 months ago

Related Issues

240

Description

This change puts quotes around values of variables (that include whitespace characters) in the teller env output

Motivation and Context

This change makes it possible to have string values that contain whitespace characters coming from the providers and be valid env variables in the output of the teller env command The existing issue is #240

How Has This Been Tested?

Added a test case in the TestTellerExports test func for the ExportDotenv code path I ran make test and it completed successfully This only affects the ExportDotenv code path, but we might want to have a similar approach for yaml

Checklist

jondot commented 4 months ago

Thanks, Teller 2.0 has a from-scratch implementation of dotenv, closing in favor of 2.0. We'll soon push the new branch