SigNoz/signoz

Move db calls to prepared statements with context

开放

#1,353 创建于 2022年7月4日

在 GitHub 查看贡献者指南
 (11 条评论) (0 个反应) (1 位负责人)TypeScript (976 个派生)batch import
backendgood first issue

仓库指标

星标
 (16,037 个星标)
PR 合并指标
 (平均合并 5天 22小时) (30 天内合并 224 个 PR)

描述

Move all db calls to prepared statements and specifically with context if possible to make signoz more secure from sql injections. A query should not be a string prepared from fmt.sprintf(...) if it has args to pass. We should try to avoid string formatting for args.

贡献者指南

研究方向
检查仓库的数据库访问层，识别所有使用字符串格式化（例如 fmt.Sprintf）构建 SQL 查询的位置。查看 issue 评论中维护者提到的具体模式。重点关注处理数据库操作的 'pkg' 或 'server' 目录中的文件。查找现有的预处理语句使用情况，以了解首选模式。考虑从一个模块开始演示方法，然后再扩展到整个代码库。
技术栈
typescript
领域
backend
议题类型
安全
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
2
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-3 小时
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
活跃
清晰度议题是否清楚说明期望改动、验收标准和下一步。
清晰
前置要求
GitTypeScript
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
70