Closed
Bug 1620682
Opened 6 years ago
Closed 6 years ago
Have process_ping_upload_request account for no-op upload attemps
Categories
(Data Platform and Tools :: Glean: SDK, enhancement, P2)
Data Platform and Tools
Glean: SDK
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: brizental, Assigned: janerik)
References
Details
(Whiteboard: [telemetry:glean-rs:m16])
Attachments
(1 file)
The current process_ping_upload_request function inside the UploadManager in glean-core requires that we send ping uuid and request status code, but there might be no status in case it wasn't even possible to attempt a request. This function needs to properly deal with that.
This first came up in: https://github.com/mozilla/glean/pull/759#discussion_r388830213
|
Reporter | |
Updated•6 years ago
|
Whiteboard: [telemetry:glean-rs:m?] → [telemetry:glean-rs:m16]
|
|
Reporter | |
Updated•6 years ago
|
Type: defect → enhancement
|
Assignee | |
Updated•6 years ago
|
Assignee: nobody → jrediger
Priority: P3 → P2
|
|
Assignee | |
Comment 2•6 years ago
|
||
We need to process the directory, no matter what. We might have pending deletion-request pings around which we still need to send.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → INVALID
|
|
Assignee | |
Comment 3•6 years ago
|
||
Closed the wrong bug, sorry.
Status: RESOLVED → REOPENED
Resolution: INVALID → ---
|
||
Comment 4•6 years ago
|
||
|
|
Assignee | |
Updated•6 years ago
|
Status: REOPENED → RESOLVED
Closed: 6 years ago → 6 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•