This message was deleted.
# contributors
b
This message was deleted.
a
This is my WIP branch: https://github.com/njculver/infracost/tree/feat/tls-configuration So far I've added flags to
cmd/infracost/run.go
, updated the
Config
struct in
internal/config/config.go
, added an
[]*Config
to the
ConfigFileSpec
struct in
internal/config/config_file.go
and I know that
doRequest
in
internal/apiclient/client.go
is where I need to add in the TLS configuration for the API calls but I think there's some intermediate plumbing that I'm still trying to sort out.
l
Thanks @agreeable-twilight-86711 🙌 . If you add a comment to the GH issue we can assign it to you. What's the reasoning for adding 
[]*Config
 to the 
ConfigFileSpec
 struct?
a
I was thinking that it could be helpful to include the TLS settings into the config file (or part of
infracost configure
) so you could specify them once in a local usage scenario rather than continuously flagging or setting environment variables.
l
Ok makes sense. The
ConfigFileSpec
is for a per-run config. I think this makes more sense as a global configuration, so we should add it to here. Initially, though I think just getting the env variables in is a great start.
👍 1