bendyworks / idkfa

Simple credentials loading
https://github.com/bendyworks/idkfa
4 stars 3 forks source link

Maintain case of idkfa keys by default #8

Open bendycode opened 11 years ago

bendycode commented 11 years ago

When confronted with an unknown key in a codebase, I ack across the project, searching for matches maintaining case sensitivity.

Currently idkfa uses all caps in ENV, and in the production code, but uses all lower case in the yml file. This would cause me to not find DEFAULT_CREDENTIALS_PATH as the place the value is maintained.

Let's make consistent case the default, or the only way idkfa works.

listrophy commented 11 years ago

Also, don't use ENV? Maybe delegate to ENV if the key exists, but in application code, the programmer should call Idkfa['foo_key'] rather than ENV['foo_key']