pufferpanel / pufferd

The PufferPanel daemon
https://pufferd.pufferpanel.com
Apache License 2.0
46 stars 15 forks source link

Camel case of "logPath" does not correspond to all other lower case config variables #59

Closed mapl closed 6 years ago

mapl commented 6 years ago

https://github.com/PufferPanel/pufferd/blob/32b8c6c5f4264c0ca8fa235d4abbb2ac56c06362/pufferd.go#L155

LordRalex commented 6 years ago

Camel casing is correct, all lowercase shouldn't really make sense. datafolder doesn't make sense to read, while dataFolder does.

I would rather that be the change, than to make it logpath. We can update our config reader to handle reading it as whatever the code says, and lowercase if we don't find it.