icestark icon indicating copy to clipboard operation
icestark copied to clipboard

关于 const namespace = 'ICESTARK' 的隐患问题

Open chuifengji opened this issue 4 years ago • 1 comments

看了下项目代码,注意到:const namespace = 'ICESTARK' ,这种写法隐患似乎大了点,我们无法预估用户代码,也无法预防潜在注入的风险,似乎应当使用 const namespace = Symbol('ICESTARK')。至于兼容性问题,我看到项目其他部分已经用到了 Symbol 内建方法,但是又有几个问题:

  1. 当前代码中多个路径(icestark-app、icestark-data 以及 src 中的 util)都分别定义了 window['ICESTARK'] 进行挂载,且处于不同的包中,这种代码结构导致无法如愿使用。。。
  2. 测试部分代码也需要修改,引入唯一定义的 const namespace = Symbol('ICESTARK')。

chuifengji avatar May 25 '21 09:05 chuifengji

这块可我们梳理下,也欢迎给我们提供 pr

maoxiaoke avatar May 25 '21 10:05 maoxiaoke