Merge pull request #1 from avireddy0/claude/gmail-scraper-cloud-run-E…#2
Closed
avireddy0 wants to merge 17 commits into
Closed
Merge pull request #1 from avireddy0/claude/gmail-scraper-cloud-run-E…#2avireddy0 wants to merge 17 commits into
avireddy0 wants to merge 17 commits into
Conversation
…mdNS Claude/gmail scraper cloud run emd ns
|
Note Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported. |
Collaborator
Author
Code reviewThis PR appears to be a stale/reversed merge PR (merging Generated with Claude Code - If this code review was useful, please react with thumbs up. Otherwise, react with thumbs down. |
|
You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool. What Enabling Code Scanning Means:
For more information about GitHub Code Scanning, check out the documentation. |
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
Gmail scraper was running as root with no .dockerignore (COPY . ./ sent entire repo to Docker daemon). Converted to multi-stage build with non-root user (uid 65532). Added .dockerignore excluding .git, tests, docs, and other non-runtime files. Constraint: functions-framework CMD preserved exactly Confidence: high Scope-risk: narrow Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
python:3.10-slim pinned to @sha256:4ba18b066cee1... in both stages. Confidence: high Scope-risk: narrow Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Org policy blocks SA key creation. Switch to Workload Identity Federation using existing github pool in claude-mcp-457317. Added id-token: write permission required for OIDC token exchange. Confidence: high Scope-risk: narrow Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Container was failing health check with failureThreshold:1. Adding startup-cpu-boost gives the container more CPU during initialization to start functions-framework before the probe timeout. Confidence: high Scope-risk: narrow Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
pip install --target=/app/deps puts binaries in /app/deps/bin/ which isn't on PATH. Container failed with 'functions-framework: not found'. Confidence: high Scope-risk: narrow Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Cloud Scheduler enforces max attempt_deadline of 30 minutes. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
…mdNS
Claude/gmail scraper cloud run emd ns