Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(设置Vercel生产环境下不使用的缓存类型): 针对特性缓存类型进行设置,不影响其他缓存方式,便于扩展 #3124

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

qixing-jk
Copy link
Contributor

@qixing-jk qixing-jk commented Jan 5, 2025

已知问题

  1. 现有Vercel缓存方式绕过方式可扩展性弱
  2. 导致Redis配置了也无法使用(虽然其实是PR的时候删除了我的REDIS_URL判断,但是多个PR提交的,所以可能觉得没用

解决方案

  1. 强制设置到不需要使用的缓存类型的调用函数内

改动收益

  1. 增强可扩展性

重要

85d6198 中的

export const isNotVercelProduction =

这次两边都定义了相同的检测变量,如果都合并的话,请保留其一,最好是 保留RSS的lib/utils/index.js中的定义,感觉可以全局使用这个参数,特别是涉及到读写文件方面的

测试确认

  • 本地开发环境测试通过
  • 生产环境构建测试通过
  • 版本号正确显示
  • 环境变量配置正常工作

Copy link

vercel bot commented Jan 5, 2025

@qixing-jk is attempting to deploy a commit to the tangly1024's projects Team on Vercel.

A member of the Team first needs to authorize it.

@qixing-jk qixing-jk force-pushed the feat-redis-cache branch 3 times, most recently from 19cef99 to fb35c44 Compare January 11, 2025 14:17
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.

1 participant