Closed Bug 1536780 Opened 5 years ago Closed 5 years ago

generic-worker: Support contentEncoding artifact property in task payload

Categories

(Taskcluster :: Workers, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: pmoore, Assigned: mraethel)

References

Details

(Keywords: good-first-bug)

Attachments

(2 files)

Generic worker already supports explicit setting of HTTP Content-Type header for task artifacts via optional task property payload.artifacts[*].contentType.

It should do the same for HTTP Content-Encoding header via optional task property payload.artifacts[*].contentEncoding.

This would allow a task author to explicitly set Content-Encoding to either identity or gzip.

I propose we support only these two values.

Existing behaviour to be preserved if property not supplied.

can I work on this bug?

Sure, Vaibhav.

Source code is available at https://github.com/taskcluster/generic-worker.

Let me know if you have questions.

Flags: needinfo?(vaibhavkaushik)

Hi Vaibhav, are you still working on this?

Many thanks,
Pete

Whiteboard: good-first-bug

Yes Pete, I am back on track & working on it again

Flags: needinfo?(vaibhavkaushik)

Thanks Vaibhav!

Blocks: 1580992
Assignee: nobody → mraethel
Status: NEW → ASSIGNED
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED

Released in generic-worker 16.2.0.

Note: testing on gecko Windows worker types in this try push.

Attachment #9097396 - Flags: review?(rthijssen)
Attachment #9097396 - Flags: review?(rthijssen) → review+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: