Stop fetching from bugzilla's rest API via model's fetch()

RESOLVED FIXED

Status

Socorro
Webapp
RESOLVED FIXED
10 months ago
10 months ago

People

(Reporter: peterbe, Assigned: peterbe)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

10 months ago
The only time SocorroCommon.fetch [0] actually makes a HTTP GET is because of the BugzillaBugInfo class [1]. Let's rid of that and just use requests "manually" there. 

Also, inside that model we have individual caching of specific cache IDs one-at-a-time so there's no benefit of using SocorroCommon's fetch method.


[0] https://github.com/mozilla/socorro/blob/1d777f0219126c4a3b7f3c8d2d9b2b5526a88642/webapp-django/crashstats/crashstats/models.py#L247
[1] https://github.com/mozilla/socorro/blob/1d777f0219126c4a3b7f3c8d2d9b2b5526a88642/webapp-django/crashstats/crashstats/models.py#L1093

Comment 2

10 months ago
Commit pushed to master at https://github.com/mozilla/socorro

https://github.com/mozilla/socorro/commit/e2c38feb71e29959e48c25a2c06f46d06784f2b9
fixes bug 1357159 - Stop using SocorroCommon.fetch for bugzilla API (#3731)

* fixes bug 1357159 - Stop using SocorroCommon.fetch for bugzilla API

* remove debugging raise

Updated

10 months ago
Status: NEW → RESOLVED
Last Resolved: 10 months ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.