Closed
Bug 1620682
Opened 5 years ago
Closed 5 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•5 years ago
|
Whiteboard: [telemetry:glean-rs:m?] → [telemetry:glean-rs:m16]
Reporter | ||
Updated•5 years ago
|
Type: defect → enhancement
Assignee | ||
Updated•5 years ago
|
Assignee: nobody → jrediger
Priority: P3 → P2
Assignee | ||
Comment 2•5 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: 5 years ago
Resolution: --- → INVALID
Assignee | ||
Comment 3•5 years ago
|
||
Closed the wrong bug, sorry.
Status: RESOLVED → REOPENED
Resolution: INVALID → ---
Comment 4•5 years ago
|
||
Assignee | ||
Updated•5 years ago
|
Status: REOPENED → RESOLVED
Closed: 5 years ago → 5 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•