Skip to content

various updates - #7

Merged
Jeffreyhung merged 2 commits into
mainfrom
bot/security-review-public-release-69fd94
Oct 21, 2026
Merged

Jeffreyhung merged 2 commits into
mainfrom
bot/security-review-public-release-69fd94

Conversation

@Jeffreyhung

Copy link
Copy Markdown
Member

No description provided.

@Jeffreyhung
Jeffreyhung merged commit 6002ae2 into main Oct 21, 2026
7 checks passed

@cursor cursor 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.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 1934700. Configure here.

Comment thread cloudbuild.yaml
logging: CLOUD_LOGGING_ONLY
substitutions:
_SERVICE_NAME: bufo-bot
_TRIGGER_ID: 475f3a5a-663f-4da0-a2f4-2fc12d839baa

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Missing trigger ID substitution

Medium Severity

Removing _TRIGGER_ID from substitutions leaves $_TRIGGER_ID in the Cloud Run --labels flag without an in-repo value. With ALLOW_LOOSE, an unsubstituted $_TRIGGER_ID is passed through, and Cloud Run rejects label values that contain $ or uppercase characters, so the deploy step can fail.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 1934700. Configure here.

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