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

RESOLVED FIXED

Status

RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: peterbe, Assigned: peterbe)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

2 years 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

2 years 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

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