Skip to content

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中的定义,感觉可以全局使用这个参数,特别是涉及到读写文件方面的

测试确认

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

@vercel
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