Skip to content

[K9VULN-16419] Add additional substring detections for proto compiler#928

Merged
jasonforal merged 1 commit into
mainfrom
jf/K9VULN-16419
Jun 23, 2026
Merged

[K9VULN-16419] Add additional substring detections for proto compiler#928
jasonforal merged 1 commit into
mainfrom
jf/K9VULN-16419

Conversation

@jasonforal

Copy link
Copy Markdown
Collaborator

What problem are you trying to solve?

Some auto generated Dart files are not being detected.

What is your solution?

Add additional substring detections:

Alternatives considered

  • Adding globs *.pb.dart, *.pbenum.dart, *.pbjson.dart, etc

Copilot AI review requested due to automatic review settings June 22, 2026 22:31
@jasonforal
jasonforal requested a review from a team as a code owner June 22, 2026 22:31
@jasonforal jasonforal changed the title Add additional substring detections for proto compiler [K9VULN-16419] Add additional substring detections for proto compiler Jun 22, 2026
@datadog-prod-us1-6

datadog-prod-us1-6 Bot commented Jun 22, 2026

Copy link
Copy Markdown

🎯 Code Coverage (details)
Patch Coverage: 100.00%
Overall Coverage: 85.01% (+0.01%)

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 90c9819 | Docs | Datadog PR Page | Give us feedback!

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR improves the static analyzer’s generated-file heuristics (in the kernel’s generated_content detector) so that auto-generated Dart files from protoc are correctly identified and excluded from analysis.

Changes:

  • Extend Dart generated-file detection with two additional header substrings commonly emitted by the Dart protobuf compiler/plugins.
  • Add unit tests covering the new Dart header variants.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@jasonforal
jasonforal merged commit f21144a into main Jun 23, 2026
108 of 109 checks passed
@jasonforal
jasonforal deleted the jf/K9VULN-16419 branch June 23, 2026 12:54
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.

3 participants