Closed Bug 1821175 Opened 2 years ago Closed 2 years ago

Push comment support is too strict when validating JSON data from Github and throwing error when sending event

Categories

(bugzilla.mozilla.org :: API, defect)

defect

Tracking

()

RESOLVED FIXED

People

(Reporter: dkl, Assigned: dkl)

References

Details

(Keywords: requested-mobile)

Attachments

(1 file)

{"code":32000,"documentation":"https:\/\/bmo.readthedocs.io\/en\/latest\/api\/","error":1,"message":"The following errors occurred when validating input data: Properties\nnot allowed: added, committer, distinct, id, modified, removed,\ntimestamp, tree_id. => \/commits\/0"}

Need to remove the ->strict portion of https://github.com/mozilla-bteam/bmo/blob/master/Bugzilla/API/V1/Github.pm#L210

Should not have been there and my tests didn't pick it up since they only use exact data in the JSON used for testing :( Will fix that as well.

Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: