Conversation
Author
|
Hello @QcloudApi @zqfan , can you review this? |
Owner
|
虽然把配置文件放到系统级能够提高便利性,使得其他用户都可以不配置就立即使用,但是也带来了危险,因为配置文件中的密钥对直接关系到现金和重要资产,配置不当权限过大的话容易造成直接的经济损失。所以这个PR可能无法接受。 |
Author
|
Hello @QcloudApi
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Use
/etc/qcloudclias config path if it exists and$HOME/.qcloudclinot exists.支持全局配置目录,对原有 HOME 下配置目录的逻辑没有影响。