$ curl -sS 'https://gerrit.wikimedia.org/r/changes/?q=616846&o=CURRENT_REVISION' | head -n 10
)]}'
[
  {
    "id": "operations%2Fsoftware%2Fwmfmariadbpy~master~I68bb5d2528c58d2e050338ae0e364df90ddf6321",
    "project": "operations/software/wmfmariadbpy",
    "branch": "master",
    "topic": "debians",
    "hashtags": [],
    "change_id": "I68bb5d2528c58d2e050338ae0e364df90ddf6321",
    "subject": "Create debian packages.",

This causes json parsing to fail.

Details

Related Changes in Gerrit:

	Subject	Author	Repo	Branch	Lines +/-
	pcc.py: fix comment about gerrit magic XSSI prefix	Jbond	operations/puppet	production	+5 -2

Customize query in gerrit

Related Objects

Duplicates Merged Here: T260963: Format error in Gerrit API response - First line starts with: )]}'

Event Timeline

• Kormat created this task.Jul 31 2020, 9:16 AM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJul 31 2020, 9:16 AM

• Kormat added a project: SRE.Jul 31 2020, 9:16 AM

• taavi added a project: Gerrit.Jul 31 2020, 9:17 AM

jbond subscribed.Jul 31 2020, 9:18 AM

Mainframe98 subscribed.Jul 31 2020, 9:21 AM

RhinosF1 subscribed.Jul 31 2020, 9:35 AM

• Kormat added a project: User-Kormat.Jul 31 2020, 10:04 AM

From https://gerrit-review.googlesource.com/Documentation/rest-api.html#output

To prevent against Cross Site Script Inclusion (XSSI) attacks, the JSON response body starts with a magic prefix line that must be stripped before feeding the rest of the response body to a JSON parser:
)]}'
[ ... valid JSON ... ]

herron triaged this task as Medium priority.Jul 31 2020, 7:00 PM

Boldly marking as invalid as this seems to be intended behavior.

Mainframe98 merged a task: T260963: Format error in Gerrit API response - First line starts with: )]}'.Aug 21 2020, 7:28 AM

Mainframe98 added subscribers: Gabrielchl, hashar.

Change 621690 had a related patch set uploaded (by Jbond; owner: John Bond):
[operations/puppet@production] pcc.py: fix comment about gerrit magic XSSI prefix

https://gerrit.wikimedia.org/r/621690

gerritbot added a project: Patch-For-Review.Aug 21 2020, 9:30 AM

Change 621690 merged by Jbond:
[operations/puppet@production] pcc.py: fix comment about gerrit magic XSSI prefix

https://gerrit.wikimedia.org/r/621690

Maintenance_bot removed a project: Patch-For-Review.Aug 21 2020, 10:10 AM

gerrit.wm.o/r/changes/ has leading garbage in the outputClosed, InvalidPublicActions

Description

Details

Related Objects

Event Timeline

gerrit.wm.o/r/changes/ has leading garbage in the output
Closed, InvalidPublic
Actions