Closed Bug 1239318 Opened 8 years ago Closed 8 years ago

ensure releases' names match in releases.name column + the blob

Categories

(Release Engineering Graveyard :: Applications: Balrog (backend), defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: bhearsum, Assigned: bhearsum)

Details

Attachments

(1 file)

Right now, Balrog allows you to insert a row into the releases table that uses one value for the "name" column, while using a different value for "name" within the release blob. This doesn't break anything within Balrog itself (the "name" in the blob is not used anywhere), but it confuses clients who request blobs through URLs like /api/releases/foo, and get back a piece of data that claims name is "bar".

We should stop allowing this inconsistency, and throw a 400 error at submission time if someone attempts to make such a change.
Attachment #8707492 - Flags: review?(catlee) → review+
Commit pushed to master at https://github.com/mozilla/balrog

https://github.com/mozilla/balrog/commit/810b29dd2992094f734ce3d0793734355c1d2eee
Merge pull request #44 from bhearsum/name-mismatch

bug 1239318: Don't allow blobs to have mismatched names between the database colum and blob data. r=catlee
Attachment #8707492 - Flags: checked-in+
This went to production today and I re-verified it there.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Product: Release Engineering → Release Engineering Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: