This repository has been archived by the owner on Nov 27, 2023. It is now read-only.
✨ Add Reverse proxy support #34
Open
Conversation
This file contains 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
|
我测试了下，会报 invalid token |
|
抱歉 是我的 Cache 部分写的有问题， 刚刚修复掉了 |
可以了。
看了下代码，可以自动刷新token。 |
gengjiawen
reviewed
Feb 27, 2023
| import { config } from "./lib"; | ||
| const getDataBasePath = () => { | ||
| if (process.env.NODE_ENV === "production") { | ||
| return "/tmp/cache.json"; |
There was a problem hiding this comment.
这个生产环境为啥不和 dev 环境保持一致，更新docker镜像就没了吧，是担心产生兼容性问题吗？
There was a problem hiding this comment.
对于一些 faas 环境，文件系统可能是只读的，所以选择了放到了 TMP 目录， 目前的 Docker 服务启动的时候应该不会主动配置 PROD， 所以这里不会触发。
gengjiawen
approved these changes
Feb 27, 2023
|
@fuergaosi233 merge了不？ |
|
悲报: 接口加了人机验证 |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
No description provided.