Skip to content

Conversation

@nnsay
Copy link
Contributor

@nnsay nnsay commented Dec 13, 2022

在提出此拉取请求时,我确认了以下几点(请复选框):

  • 我已阅读并理解贡献者指南
  • 我已检查没有与此请求重复的拉取请求。
  • 我已经考虑过,并确认这份呈件对其他人很有价值。
  • 我接受此提交可能不会被使用,并根据维护人员的意愿关闭拉取请求。

填写PR内容:

  • 支持从配置文件中获取public key.

@eryajf
Copy link
Member

eryajf commented Dec 13, 2022

很棒的改动,不过我看到这个key,还有另外几个地方在用到,麻烦你也一并改成变量引用的方式吧

@eryajf
Copy link
Member

eryajf commented Dec 13, 2022

另外我试了下,放在配置文件中,不加换行符也是可以直接使用的,将秘钥内容合并成一行,也可以。

VUE_APP_PUBLIC_KEY="-----BEGIN PUBLIC KEY-----MIGfMA0GCSqGSIb3DQEBAQUAA4GNADCBiQKBgQC7DSaq24RM3FdYQumg7DQPpif8sS6HOjtNAGhUN8oW9Gh4aIRZICxUi3RMywGZPZJiw6hAgnEKBYr3YbqcMgiaMvybyUCBR8tjtNx+vh8dg8b/5C/jmjk5tQg9bKG78LF+xdpbWIi/5RvMP8k6S2DID/z97s9yW8d3K8j/IUWhgwIDAQAB-----END PUBLIC KEY-----"

@eryajf
Copy link
Member

eryajf commented Dec 13, 2022

当这个配置项提取到了配置文件中以后,就不需要原来的的了,所以可以直接替换掉,而不用 process.env.VUE_APP_PUBLIC_KEY || oldkey 这样子。 直接用 process.env.VUE_APP_PUBLIC_KEY 替换好了。

@nnsay nnsay force-pushed the feature/publickey-config branch from abc4e2d to 9e48bfb Compare December 13, 2022 06:41
@jianwang-beijing
Copy link

当这个配置项提取到了配置文件中以后,就不需要原来的的了,所以可以直接替换掉,而不用 process.env.VUE_APP_PUBLIC_KEY || oldkey 这样子。 直接用 process.env.VUE_APP_PUBLIC_KEY 替换好了。

使用证书字符串的地方都修改为引用这个变量了

@eryajf eryajf merged commit 0664b3a into opsre:main Dec 13, 2022
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

Successfully merging this pull request may close these issues.

3 participants