John Eikenberry
John Eikenberry
First let me say thanks for DMA, it's a great lightweight MTA. I use it everywhere I don't want a full MTA and it does the job perfectly. I'm trying...
### What is the URL of the page with the issue? https://pkg.go.dev/encoding/pem ### What is your user agent? Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Firefox/91.0 ### Screenshot ![Screenshot_from_2022-06-23_18-32-30](https://user-images.githubusercontent.com/722663/175442716-00f3d5a2-00ac-42fc-96ed-7583b07cf709.png) ### What...
Template execute errors would return the values related to the error in certain cases and these could end up in the logs. This makes sure that if those values are...
In VaultConfig's GoString() method it references `*c.LeaseRenewalThreshhold`, but `c` can be `nil` and, if so, this panics. It should use a `FloatGoString()` method (similar to the others in config/convert.go) to...
Some of the template functions have overly specific input parameters. Like `map[string]interface{}` instead of just `interface{}`. This restricts their usefulness unnecessarily. Review the template function input parameters and fix this....
Related to: https://github.com/hashicorp/consul-terraform-sync/pull/164 Review the defaults (if any) and consider applying the defaults CTS uses. These are the default values changed in CTS, as copied from Consul-template. See that ticket...
Both `parseInt` and `parseFloat` take strings as arguments and convert them to 64 bit numbers of their appropriate type. The json.Number type is a string sub-type that contains integers or...
Both Vault and Consul blocks support a `transport` block with various settings.. https://github.com/hashicorp/consul-template/blob/f87266ee0c36f4b57b8a62a694e454f9475b1611/config/transport.go#L38-L62 Need to add config documentation for these.
Document, probably in the Caveats->Command section, the case where if you have multiple templates with commands and they all have the same command (strings match) that command will only be...
Currently consul-template only supports a single Namespace for Consul. It should be able to work with multiple Namespaces as long as it has access. Consul template functions will probably need...