Skip to content
This repository has been archived by the owner on Jul 26, 2023. It is now read-only.

增加更多config的环境变量 #530

Open
caspershw opened this issue Mar 8, 2022 · 4 comments
Open

增加更多config的环境变量 #530

caspershw opened this issue Mar 8, 2022 · 4 comments
Labels
feature request New feature or request

Comments

@caspershw
Copy link

目前似乎只有userPrincipalName这一项可以使用环境变量,希望api.config.js和site.config.js里面大部分配置都可以使用环境变量的方式部署,也可以避免潜在泄露信息的风险。

再提个小tips,可以使用 https://github.com/apps/pull 这个小工具实现对上游更新的自动同步,而且通过环境变量设置,无需改动文件,以后也可以避免自动同步时出现冲突~

@spencerwooo spencerwooo added the feature request New feature or request label Mar 9, 2022
@spencerwooo
Copy link
Owner

虽然但是,后面想把这些配置项目抽象到 Redis 数据库里面,这样还可以实现一个后台 admin panel 类似的样子。也可以考虑用环境变量的形式,以后再说。

@Android-KitKat
Copy link
Contributor

Android-KitKat commented Mar 9, 2022

虽然但是,后面想把这些配置项目抽象到 Redis 数据库里面,这样还可以实现一个后台 admin panel 类似的样子。也可以考虑用环境变量的形式,以后再说。

感觉可以,如果配置和项目文件分离,Fork 的仓库就能直接 Fetch upstream,更新起来确实方便不少。

@chisaato
Copy link
Contributor

那这样的话,再通过环境变量指定一个前缀呗. 有前缀以后就可以在一个 Redis 数据库下支持多个实例了.
因为 upstart 免费只支持一个数据库 而一个免费数据库的容量开几个index都没问题

@iRedScarf
Copy link

iRedScarf commented Jun 21, 2023

设置成环境变量确实会很方便,至少可以很便捷地修改要分享目录对吧,可以很便捷地修改密码什么的。
我有在尝试改成一键部署(首次部署时在环境变量中设置site.config和api.config的一些信息),但是我不是很懂(全靠问GPT),稍微调整了一些之后,在OAuth第一步的时候就没办法获取到CLIENT_ID和CLIENT_SECRET的值,啊……
我成功了,把ClientId、ClientSecret、分享目录、网页title、OneDrive帐户都放在环境变量中设置了。
晚点我休整下代码再推给你看看?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature request New feature or request
Projects
None yet
Development

No branches or pull requests

5 participants