abcxyz / abcxyz/github-metrics-aggregator

Add validation to SQL inputs for webhook and retry service.

Open
#199 0 comments 0 reactions 0 assignees View on GitHub
good first issue
Dominant language
Go
Stars
16
Forks
3
PR merge metrics
No merged PRs in 30d

Description

`pkg/retry` and `pkg/webhook` doesn't yet use the improved validation added in #198

This is not a huge concern as the threat model for this particular injection risk is a privileged insider who likely already has sufficient access to the database anyways. Mainly this is to ensure that we have secure patterns so if they are copied or used elsewhere we don't accidentally introduce injection bugs.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.