Giter VIP home page Giter VIP logo

Comments (12)

harrisjb avatar harrisjb commented on June 10, 2024 1

After review I believe this is expected behavior as the gem is just trying to read from ENV namespace if things are loaded into ENV by Heroku or AWS or some other means. It doesn't intend to write environment variables into the ENV namespace itself, it's just reading from it.

from config.

harrisjb avatar harrisjb commented on June 10, 2024

Similar issue here:

  • config 4.0.0
  • ruby 2.5.8
  • rails 5.1.7

Contents of config/initializers/config.rb

# set up config to look for ENV[key] instead of Settings.key
Config.setup do |config|
  config.use_env = true
end    

rails console does not load into ENV namespace:

$ rails c
Loading development environment (Rails 5.1.7)
[1] pry(main)> ENV['database_host']
=> nil
[2] pry(main)> Settings.database_host
=> "localhost"

from config.

pkuczynski avatar pkuczynski commented on June 10, 2024

Yes @harrisjb we do not populate things back into ENV, we only read from it. So your reported issue is not valid.

from config.

pkuczynski avatar pkuczynski commented on June 10, 2024

However what @deepakmahakale posted should work. @cjlarose any idea why it does not?

from config.

cjlarose avatar cjlarose commented on June 10, 2024

I agree that what @deepakmahakale posted should work. @deepakmahakale if you're able to reproduce the issue on a new rails app, we can take a deeper look into what's going on.

from config.

jfadgen avatar jfadgen commented on June 10, 2024

I am having the same issue as @deepakmahakale , in a new Rails app:

  • config 4.0.0
  • rails 7.0.3
  • ruby 2.7.4
Config.setup do |config|
  config.use_env = true
  config.env_prefix = 'CONFIG'
  config.env_separator = '__'
  config.env_converter = :downcase
  config.env_parse_values = true
end

.env

CONFIG__COGNITO__USER_POOL_ID="test_pool_id"
CONFIG__COGNITO__CLIENT_ID="test_client_id"
CONFIG__COGNITO__CLIENT_SECRET_KEY="test_secret_key"
CONFIG__ANYTHING_TEST="anything at all"
irb(main):001:0> ENV["CONFIG__COGNITO__USER_POOL_ID"]
=> "test_pool_id"
irb(main):002:0> ENV["CONFIG__COGNITO__CLIENT_ID"]
=> "test_client_id"
irb(main):003:0> ENV["CONFIG__COGNITO__CLIENT_SECRET_KEY"]
=> "test_secret_key"
irb(main):004:0> ApplicationConfig.cognito
=> nil
irb(main):005:0> ENV["CONFIG__ANYTHING_TEST"]
=> "anything at all"
irb(main):006:0> ApplicationConfig.anything_test
=> nil

I tried downgrading to config gem 2.2, but without luck. I have another project using that version (with Rails 6.1.4.1, Ruby 2.7.4.) successfully.

Happy to try anything if you have any recommendations, thank you.

from config.

nick-reshetnikov avatar nick-reshetnikov commented on June 10, 2024

@jfadgen, @deepakmahakale, I ran into a similar issue in a recent rails project and figured out that the problem is that ENV variables haven't been loaded yet when the config initializer runs if you're using .env files.

specifically, if you're loading from a .env file, you will likely need load the .env files before the initializers run.

according to https://github.com/bkeepers/dotenv:

dotenv is initialized in your Rails app during the before_configuration callback, which is fired when the Application constant is defined in config/application.rb with class Application < Rails::Application. If you need it to be initialized sooner, you can manually call Dotenv::Railtie.load.

# config/application.rb
Bundler.require(*Rails.groups)

# Load dotenv only in development or test environment
if ['development', 'test'].include? ENV['RAILS_ENV']
  Dotenv::Railtie.load
end

note that if you're using something like heroku, where the env vars are set on the servers, they will likely be available for the initializers. so you may only need to do this for your dev and test environments.

@pkuczynski I think you guys should think about adding support for .env files - if they're detected, your gem should automatically try to load the contents of .env before config/initializers/config.rb runs

from config.

pkuczynski avatar pkuczynski commented on June 10, 2024

The whole concept of this gem is to store config in the per env configuration files. Using .env and gems like dotenv is a different approach for doing the same thing. They kind of contradictory to each other. So I am not sure if we should support this. @cjlarose what do you think?

from config.

nick-reshetnikov avatar nick-reshetnikov commented on June 10, 2024

yeah, I see what you're saying, but the issue is that a lot of teams use .env files for dev/test environments, but manage environment variables directly on staging/prod servers (e.g. thru Heroku config vars). pretty much every project I've worked on has used this hybrid approach. I honestly haven't seen .env.production really used, certainly not with things like heroku.

so, you're kind of leaving the door open to a lot of confusion if you don't support env vars being preloaded thru something like dotenv, which is a very popular tool.

for me, I wanted to use this gem primarily to store certain configs in a yml file format, which allows for nesting keys and other advantages over long, messy .env files. however, I also wanted to be able to override default values in that yml file thru env vars (i.e. on heroku). this allows me to change these configs without committing new yml files. I think that's a legitimate use case.

but in any case, you guys should at least add a disclaimer about this scenario in the README, since I'm sure others have ran into this issue

from config.

jfadgen avatar jfadgen commented on June 10, 2024

Worth noting that the issue went away for me after changing my Gemfile to bundle
gem "dotenv-rails"
instead of
gem "dotenv".

from config.

nick-reshetnikov avatar nick-reshetnikov commented on June 10, 2024

@jfadgen I in fact saw this issue while using "dotenv-rails"

from config.

nick-reshetnikov avatar nick-reshetnikov commented on June 10, 2024

actually, I just discovered that you can require dotenv/rails-now (https://github.com/bkeepers/dotenv/blob/master/lib/dotenv/rails-now.rb) in your gemfile to solve the issue:

gem 'dotenv-rails', require: 'dotenv/rails-now'

this will ensure that dotenv files are loaded before initializers and solve this issue. I think modifying the gemfile is preferable to messing with application.rb in the manner I showed above.

from config.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.