Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Possible recursive expansion regression #117

Closed
kioan000 opened this issue Feb 13, 2024 · 4 comments
Closed

Possible recursive expansion regression #117

kioan000 opened this issue Feb 13, 2024 · 4 comments

Comments

@kioan000
Copy link

kioan000 commented Feb 13, 2024

After upgrading from 10.0 to latest 11.0.3

the following .env

BACKEND_API_HEALTH_CHECK_URL=${MOCK_SERVER_HOST}/ci-health-check
...
MOCK_SERVER_HOST=http://localhost:${MOCK_SERVER_PORT}
MOCK_SERVER_PORT=8090

parsed by this script

expand(dotenv.config({ path: path.resolve(root, '.env') }))

console.log('env', process.env)

is not recursively expanded anymore. Now it produces the following result:

BACKEND_API_HEALTH_CHECK_URL: 'http://localhost:${MOCK_SERVER_PORT}/ci-health-check',
...
MOCK_SERVER_HOST: 'http://localhost:8090',
MOCK_SERVER_PORT: '8090',

@kioan000 kioan000 changed the title Recursive expansion regression Possible recursive expansion regression Feb 13, 2024
@motdotla
Copy link
Owner

looking into this now

@motdotla
Copy link
Owner

effort happening here: #118

@motdotla
Copy link
Owner

something you can do to fix this for yourself right now is to place BACKEND_API_HEALTH_CHECK_URL at the end of your .env file. that will expand correctly. 67fcfee#diff-2ab7c3c5008dfeaf54c1692893b6e2677fe5b9afc15935d7c94e515f383deab3R516

@motdotla
Copy link
Owner

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants