Skip to content

fix(google-indexing): treat 403 PERMISSION_DENIED as warning, not failure - #21

Draft
samtrion with Copilot wants to merge 1 commit into
mainfrom
copilot/fix-google-indexing-submit-check
Draft

fix(google-indexing): treat 403 PERMISSION_DENIED as warning, not failure#21
samtrion with Copilot wants to merge 1 commit into
mainfrom
copilot/fix-google-indexing-submit-check

Conversation

Copilot AI commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

The Google Indexing workflow was hard-failing CI on every page_build event because the service account lacks verified ownership of daily-devops.net in Search Console, causing the API to return HTTP 403. Since this is a configuration gap (not a transient or code error), exiting with code 1 is wrong.

Changes

  • New warned bucket for 403 responsesPERMISSION_DENIED errors are logged as WARN and tracked separately; only non-403 failures (5xx, 429, etc.) trigger sys.exit(1)
  • Step summary updated — 403 entries appear as [!] warnings instead of [ ] failures, making the distinction visible in the job summary

To restore actual indexing: add the service account email from GOOGLE_INDEXING_SA_KEY as a verified owner in Google Search Console for daily-devops.net.

…failure

Co-authored-by: samtrion <3283596+samtrion@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants