uioz / mfe-proxy-cli

MIT License
0 stars 0 forks source link

# 建议将 `mfe-route.json` 与 `mfe-config.js` 进行合并 #8

Open uioz opened 3 years ago

uioz commented 3 years ago

建议将 mfe-route.jsonmfe-config.js 进行合并

why

  1. 减少配置文件数量
  2. 可以使用 JS 语法更加灵活
  3. 使用 commonjs 语法可以被其他模块引用, 减少冗余

缺点

  1. mfe-proxy-middleware 解析该文件比较困难
  2. mfe-proxy-server 要实现远程代理应用比较困难(目前放弃远程方案)
  3. mfe-config.js 不能引入任何模块, 而且服务器托管的 mfe-config.js 所依赖的执行环境必须与当前执行该脚本环境兼容.
  4. mfe-config.js 将不能被公开访问, 防止敏感信息泄漏

解析方案

  1. mfe-proxy-middleware 将远程脚本下载到本地后直接使用 node 解析
  2. mfe-proxy-server 则直接读取数据
  3. mfe-proxy-server 开启远程模式的情况下, 使用现有的 mfe-route.json 方案(目前放弃远程方案)

Originally posted by @uioz in https://github.com/uioz/mfe-proxy-cli/issues/2#issuecomment-823854412