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)

enhancement

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

Whiteboard: [telemetry:glean-rs:m?] → [telemetry:glean-rs:m16]
Type: defect → enhancement
Assignee: nobody → jrediger
Priority: P3 → P2

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

Closed the wrong bug, sorry.

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

Attachment

General

Created:
Updated:
Size: