eggjs / eggjs/egg

配置中的属性能否支持上symbol

Open
#5,175 14 comments 0 reactions 0 assignees View on GitHub
Dominant language
TypeScript
Stars
19k
Forks
1.8k
PR merge metrics
No merged PRs in 30d

Description

### 请详细告知你的新点子(Nice Ideas):

现状:
在配置文件中以symbol作为属性自定义配置值后,通过app.config访问会丢失这些配置值。

期望实现:
在配置文件中,能够以symbol作为属性,做一些自定义的配置。(看了一圈issue好像没有人提到过

理由:
目前我使用的业务框架继承自egg,内部集成了很多插件和相关的配置。在写业务的时候担心写自定义配置时,因使用了重复的属性名导致配置冲突所以考虑使用symbol作为属性。

能够做些什么:
配置的合并使用了extend2模块进行深度拷贝,看了下源码没有对symbol值做拷贝处理所以导致symbol丢失。我自己参考lodash的深拷贝方法在extend2模块上添加了symbol值的拷贝处理后实现了这个功能。想问下大佬这里是否可以给extend2模块提个pr然后让egg更新依赖版本支持上这个功能。

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.