Skip to content

Authenticate googlesource.com requests behind a feature flag - #5547

Merged
PauloVLB merged 1 commit into
masterfrom
gob-auth-source
Oct 22, 2026
Merged

PauloVLB merged 1 commit into
masterfrom
gob-auth-source

Conversation

@PauloVLB

@PauloVLB PauloVLB commented Oct 20, 2026 •

Copy link
Copy Markdown
Collaborator

b/573204199

Problem

Anonymous requests to googlesource.com (GoB) are no longer covered by its SLO and are the first ones dropped when it's overloaded. We fetch DEPS files and commit info from chromium.googlesource.com without auth for the revision info in fuzzer logs, impact, blame and the testcase page (~16k requests a day).

Proposed solution

  • Behind a new gob_authenticated_requests feature flag, googlesource.com requests go to the /a/ path with the default service account token.
  • GoB accepts cloud-platform scoped tokens, which the bots and App Engine already have, so there's nothing to provision.

Validation

In dev, with the flag on:

  • I opened /revisions?job=libfuzzer_chrome_asan&revision=1723778, which fetches that revision's DEPS from chromium.googlesource.com. The logs show the request going to the /a/ URL with the App Engine service account, and the page returned 200 with the component revisions:
    image

  • On a dev bot, an impact task made the authenticated calls with the bot's service account, and they worked: right after each one it parsed the DEPS and V8 commit info (logs):
    image

@PauloVLB
PauloVLB force-pushed the gob-auth-source branch 3 times, most recently from 9fb3249 to 17b4d85 Compare October 21, 2026 15:23
@PauloVLB
PauloVLB marked this pull request as ready for review October 21, 2026 15:23
@PauloVLB
PauloVLB requested a review from a team as a code owner October 21, 2026 15:23
@PauloVLB
PauloVLB requested a review from dylanjew October 21, 2026 15:23
@PauloVLB PauloVLB assigned decoNR and unassigned decoNR Oct 21, 2026
@PauloVLB
PauloVLB requested a review from decoNR October 21, 2026 15:23

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

LGTM, please wait your test in dev to merge.

@PauloVLB
PauloVLB merged commit 11c52db into master Oct 22, 2026
15 checks passed
@PauloVLB
PauloVLB deleted the gob-auth-source branch October 22, 2026 18:40
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