Hacktoberfest 2026:维护者为十月标记出来的 issue,仍然开放、适合新手。 浏览 Hacktoberfest issue

perf: limit cookie parse only on some cookie

未关闭
#126 0 条评论 1 个 reaction 已指派 0 人 在 GitHub 查看

@cesco69 已经在做这个了。

开始于 2025年4月15日。

  • #127 来自 @cesco69 —— 未关闭

评估

难度
3/5
预计耗时
1-2 天
新手友好度
42/100
Issue 类型
功能
描述清晰度
基本清楚
活跃度
停滞
领域
backend

调研方向

从 cookieParser 中间件的入口点开始,检查请求 Cookie 标头的解析方式。检查现有的选项相关测试,然后将完成标准定义为:提供一个将解析限制为选定 Cookie 的选项,并添加一个覆盖 API_TOKEN 示例的回归测试。

由索引模型根据 Issue 内容生成。

描述

Hi, in my express I use only 1 cookie (API_TOKEN), but client side send varius cookie eg:

_ga_M9MSXKVJN9=XXXX; 
_ga_W98228BE06=XXXX; 
_gid=XXXX; _ga=XXXX; 
_gcl_au=XXXX; 
_ga_5MFMLYZHT2=XXXX; 
rxVisitor=XXXX; 
_ga_K4Y50QD36K=XXXX; 
dtCookie=XXXX; 
dtSa=XXXX; 
rxvt=XXXX; 
API_TOKEN=XXXX;

the _g* cookies are Google Analitycs / Google Tag Manage
the _dt* and rx* cookies are Dynatrace
the API_TOKEN is the only one used by my express!

Parsig all those cookies on each request has a performance impact!

The idea is introduce an option for limit the cookie parsing on a limit numer of cookies, eg.:

app.use(cookieParser({null, {
   allowedCookie: ['API_TOKEN']
}}))
主要语言
JavaScript
星标
2k
派生
236
PR 合并指标
30 天内没有已合并 PR

环境准备

  • 没有 Dockerfile 或 Docker Compose 文件
  • 没有 Pull Request 模板
  • 阅读贡献指南

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

expressjs/cookie-parser 的其他 Issue

查看 expressjs/cookie-parser 的全部 Issue

相似的 Issue

更多 JavaScript Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。