# 安全审查工作流程 ## 何时使用此技能 * 代码变更的安全审计 * 实现认证/授权 * 数据保护审查 * 漏洞评估 ## 飞行前检查清单 * 识别安全敏感区域 * 回顾 OWASP Top 10 相关性 * 检查现有的安全模式 * 计划修复方法 ## OWASP 十大清单 ### 1\.访问控制损坏 // :x: 存在漏洞 - 没有授权检查 [HttpGet("{id}")] 公共异步任务 <员工> Get(字符串 id) => 等待 repo.GetByIdAsync(id); // :white_check_mark: 安全 - 强制授权 [HttpGet("{id}")] [平台授权（角色.Manager，角色.管理员）] 公共异步任务 <员工> Get(字符串 id) { var 员工 = 等待 repo.GetByIdAsync(id); // 验证对此特定资源的访问 if (employee.CompanyId != RequestContext.CurrentCompanyId()) 抛出新的 UnauthorizedAccessException(); 返回员工； } ### 2\.加密失败 // :x: 存在漏洞 - 存储纯文本机密 var apiKey = config["ApiKey"]; 等待 SaveToDatabase(apiKey); // :white_check_mark: SECURE - 加密敏感数据 var 加密密钥 = cryptoService.Encrypt(apiKey); 等待 SaveToDatabase(加密密钥); // 使用安全配置 var apiKey = config.GetValue("ApiKey"); // 来自 Azure 密钥保管库 ### 3\.注射 // :x: 存在漏洞 - SQL 注入 var sql = $"从用户中选择 * WHERE Name = '{name}'"; 等待 context.Database.ExecuteSqlRawAsync(sql); // :white_check_mark: SECURE - 参数化查询等待 context.Users.Where(u => u.Name == name).ToListAsync(); // 或者如果需要原始 SQL：等待上下文.Database.ExecuteSqlRawAsync( “从用户中选择*，其中名称 = @p0”，名称）； ### 4\.不安全的设计 // :x: 脆弱 - 无速率限制 [HttpPost("登录")] 公共异步任务登录（LoginRequest 请求） => 等待 authService.Login(请求); // :white_check_mark: 安全 - 应用速率限制 [HttpPost("登录")] [RateLimit(最大请求数 = 5, WindowSeconds = 60)] 公共异步任务登录（LoginRequest 请求） => 等待 authService.Login(请求); ### 5\.安全配置错误 // :x: 脆弱 - 生产中的详细错误 app.UseDeveloperExceptionPage(); // 暴露堆栈跟踪 // :white_check_mark: SECURE - 生产中的一般错误 if (env.IsDevelopment()) app.UseDeveloperExceptionPage(); 否则 app.UseExceptionHandler("/Error"); ### 6\.易受攻击的组件 # 检查是否存在易受攻击的包 dotnet list 包--有漏洞 # 更新有漏洞的包点网已过时 ### 7\.认证失败 // :x: 脆弱 - 弱密码策略 if (密码.长度 >= 4) { } // :white_check_mark: 安全 - 强密码策略公共类密码策略 { 公共 bool 验证（字符串密码） { 返回密码。长度 >= 12 && 密码.Any(char.IsUpper) && 密码.Any(char.IsLower) && 密码.Any(char.IsDigit) && 密码.Any(c => !char.IsLetterOrDigit(c)); } } ### 8\.数据完整性故障 // :x: 存在漏洞 - 未验证外部数据 var userData =等待externalApi.GetUserAsync(id); 等待 SaveToDatabase(userData); // :white_check_mark: SECURE - 验证外部数据 var userData =等待externalApi.GetUserAsync(id); var 验证 = userData.Validate(); if (!validation.IsValid) 抛出新的 ValidationException(validation.Errors); 等待 SaveToDatabase(userData); ### 9\.记录失败 // :x: 存在漏洞 - 记录敏感数据 Logger.LogInformation("用户登录：{邮箱} {密码}",邮箱,密码); // :white_check_mark: SECURE - 编辑敏感数据 Logger.LogInformation("用户登录：{Email}", email); // 切勿记录密码、令牌或 PII ### 10\. SSRF（服务器端请求伪造） // :x: 存在漏洞 - 用户控制的 URL var url = request.WebhookUrl; 等待 httpClient.GetAsync(url); // 可以访问内部服务 // :white_check_mark: SECURE - 验证和限制 URL if (!IsAllowedUrl(request.WebhookUrl)) throw new SecurityException("无效的 webhook URL"); 私有 bool IsAllowedUrl(字符串 url) { var uri = 新的 uri(url); 返回AllowedDomains.Contains(uri.Host) && uri.Scheme == "https"; } ## 授权模式 ### 控制器级别 [API控制器] [路线(“api/[控制器]”)] [PlatformAuthorize] // 需要身份验证公共类 EmployeeController : PlatformBaseController { [http邮报] [PlatformAuthorize(Roles.Admin, Roles.Manager)] // 基于角色公共异步任务 Create(...) } ### 处理程序级别受保护的覆盖异步任务 > ValidateRequestAsync( PlatformValidationResult 验证，CancellationToken ct) { 返回等待验证 // 检查角色 .And(_ => RequestContext.HasRole(Roles.Admin), "需要管理员角色") // 检查公司访问权限 .And(_ => 实体.CompanyId == RequestContext.CurrentCompanyId(), “访问被拒绝：不同的公司”） // 检查所有权 .And(_ => 实体.OwnerId == RequestContext.UserId() || RequestContext.HasRole(角色.Admin), “访问被拒绝：不是所有者”）； } ### 查询级别 // 始终按公司/用户上下文过滤 var 员工 = 等待 repo.GetAllAsync( e => e.CompanyId == RequestContext.CurrentCompanyId() && (e.IsPublic || e.OwnerId == RequestContext.UserId())); ## 数据保护 ### 敏感数据处理公共类 SensitiveDataHandler { // 静态加密公共字符串EncryptForStorage（字符串纯文本） => 加密服务.Encrypt(纯文本); // 显示掩码公共字符串MaskEmail（字符串电子邮件） { var parts = email.Split('@'); 返回$“{parts[0][0]}***@{parts[1]}”; } // 切勿记录敏感数据公共无效LogUserAction（用户用户） { Logger.LogInformation("用户操作：{UserId}", user.Id); // NOT: Logger.Log("用户: {Email} {Phone}", user.Email, user.Phone); } } ### 文件上传安全公共异步任务上传（IFormFile 文件） { // 验证文件类型 var allowedTypes = new[] { ".pdf", ".docx", ".xlsx" }; var 扩展名 = Path.GetExtension(file.FileName).ToLowerInvariant(); if (!allowedTypes.Contains(扩展名)) return BadRequest("无效的文件类型"); // 验证文件大小 if (文件.长度 > 10 * 1024 * 1024) // 10MB return BadRequest("文件太大"); // 扫描恶意软件（如果有） if (!await antivirusService.ScanAsync(文件)) return BadRequest("文件被安全扫描拒绝"); // 生成安全文件名 var safeFileName = $"{Guid.NewGuid()}{扩展名}"; // 保存到独立存储等待 fileService.SaveAsync(文件, safeFileName); 返回确定（）； } ## 安全扫描命令 # .NET 漏洞扫描 dotnet list 包--有漏洞 # 过时的软件包点网已过时 # 秘密扫描 grep -r "密码\|秘密\|apikey" --include="*.cs" --include="*.json" # 硬编码凭证 grep -r "密码=\"" --include="*.cs" grep -r "connectionString.*password" --include="*.json" ## 安全审查清单 ### 身份验证 * 强制执行强密码策略 * 尝试失败后帐户被锁定 * 安全会话管理 * JWT 令牌经过正确验证 * 刷新令牌轮换 ### 授权 * 所有端点都需要身份验证 * 实施基于角色的访问控制 * 检查资源级权限 * 无法提升权限 ### 输入验证 * 所有输入均已验证 * 防止 SQL 注入（参数化查询） * 防止 XSS（输出编码） * 文件上传已验证 * 重定向的 URL 验证 ### 数据保护 * 静态加密的敏感数据 * 强制执行 HTTPS * 日志中没有敏感数据 * 正确的错误处理（无堆栈跟踪） ### 依赖关系 * 没有已知的易受攻击的软件包 * 依赖项定期更新 * 第三方代码审核 ## 要避免的反模式 :x: **信任客户端输入** var isAdmin = request.IsAdmin; // 用户提供的！ :x: **暴露内部错误** catch (异常 ex) { return BadRequest(ex.ToString()); } :x: **硬编码的秘密** var apiKey = "sk_live_xxxxx"; :x: **日志记录不足** // 敏感操作没有审计跟踪等待删除所有用户（）； ## 验证清单 * OWASP 前 10 名已审核 * 验证/授权验证 * 输入验证完成 * 敏感数据受到保护 * 没有硬编码的秘密 * 适当记录（无 PII） * 扫描依赖关系

Arch 安全审查

包含技能