Skip to content

Add actionResult mapping for GCP DNS and HTTP request logs - #2404

Open
JocLRojas wants to merge 1 commit into
v11from
backlog/gcp-filter-action-result
Open

Add actionResult mapping for GCP DNS and HTTP request logs#2404
JocLRojas wants to merge 1 commit into
v11from
backlog/gcp-filter-action-result

Conversation

@JocLRojas

Copy link
Copy Markdown
Contributor

What this PR does

Adds actionResult mapping in the GCP filter for two log families that were previously indexed without that field:

  • Cloud DNS query logs (dns.googleapis.com/dns_queries): maps jsonPayload.responseCode to actionResult:
  • NOERROR → success
  • REFUSED → denied
  • any other response code (NXDOMAIN, SERVFAIL, FORMERR, NOTIMP, ...) → failed

Cloud Run / Load Balancer / App Engine / Cloud Functions HTTP request logs (which carry httpRequest.status renamed as statusCode):

  • 1xx–3xx → success
  • 401 and 403 → denied (access denial, distinct from generic failure)
  • remaining 4xx and 5xx → failed

Related issue

N/A

@JocLRojas
JocLRojas requested a review from a team July 28, 2026 20:29
@github-actions

Copy link
Copy Markdown

❌ Go dependencies check failed

There are outdated Go dependencies, or modules that could not be inspected.
Run bash .github/scripts/go-deps.sh --update --discover locally and
commit the updated go.mod / go.sum files.

Script output
🔍 Discovered 26 Go projects

📦 Dependencies with updates available:

  📁 ./plugins/aws:
     - github.com/aws/aws-sdk-go-v2: v1.43.0 → v1.43.1
     - github.com/aws/aws-sdk-go-v2/config: v1.32.31 → v1.32.32
     - github.com/aws/aws-sdk-go-v2/credentials: v1.19.30 → v1.19.31
     - github.com/aws/aws-sdk-go-v2/service/cloudwatchlogs: v1.80.0 → v1.80.1

  📁 ./plugins/modules-config:
     - github.com/aws/aws-sdk-go-v2/config: v1.32.31 → v1.32.32
     - github.com/aws/aws-sdk-go-v2/credentials: v1.19.30 → v1.19.31
     - github.com/aws/aws-sdk-go-v2/service/cloudwatchlogs: v1.80.0 → v1.80.1
     - github.com/aws/aws-sdk-go-v2/service/sts: v1.45.0 → v1.45.1

�[0;31m❌ Please update dependencies before merging.�[0m

@github-actions

Copy link
Copy Markdown

✅ AI review — Approved

No issues detected in this diff.

architecture (gemini-3-flash-lite) — clean

Summary: No reviewable changes in this diff (excluded paths only).

No findings.

bugs (gemini-3-flash-lite) — clean

Summary: No reviewable changes in this diff (excluded paths only).

No findings.

security (gemini-3-flash-lite) — clean

Summary: No reviewable changes in this diff (excluded paths only).

No findings.

@utmstackprapprover utmstackprapprover Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Changes requested — Go dependencies check failed (see above).

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.

1 participant