nuxt-community/dotenv-module

System vars should overwrite dotenv vars

Open

#46 创建于 2019年10月15日

在 GitHub 查看
 (9 评论) (3 反应) (0 负责人)JavaScript (482 star) (32 fork)batch import
cmty:feature-requesthelp wanted

描述

What problem does this feature solve?

In most places where dotenv is used system env vars override the dotenv vars, however dotenv-module does the exact opposite, swapping this around would give the expected result when using systemvars: true

贡献者指南

技术栈
javascriptnodejs
领域
backend
议题类型
feature
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
2
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
under 1 hour
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
stale
清晰度议题是否清楚说明期望改动、验收标准和下一步。
clear
前置要求
Basic Node.js knowledgeFamiliarity with dotenv
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
85
研究方向
Investigate the dotenv module source code to find where environment variables are loaded and the merging logic. The issue is in the module's core file (likely `lib/module.js` or similar) where `process.env` and dotenv parsed values are combined. The fix is to ensure that system environment variables take precedence over dotenv values when `systemvars: true` is set. Review the existing tests and add a test case for this behavior.