generic-worker: Support contentEncoding artifact property in task payload
Categories
(Taskcluster :: Workers, enhancement)
Tracking
(Not tracked)
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.
Comment 1•5 years ago
|
||
can I work on this bug?
Reporter | ||
Comment 2•5 years ago
|
||
Sure, Vaibhav.
Source code is available at https://github.com/taskcluster/generic-worker.
Let me know if you have questions.
Reporter | ||
Comment 3•5 years ago
•
|
||
Note, the link in comment 0 is no longer valid, but the payload schemas can be seen in raw here:
https://github.com/taskcluster/generic-worker/blob/v14.1.0/native_all-unix-style.yml#L107-L121 (mac, linux)
https://github.com/taskcluster/generic-worker/blob/v14.1.0/native_windows.yml#L111-L124 (windows)
Reporter | ||
Comment 4•5 years ago
|
||
Hi Vaibhav, are you still working on this?
Many thanks,
Pete
Updated•5 years ago
|
Reporter | ||
Updated•5 years ago
|
Comment 5•5 years ago
|
||
Yes Pete, I am back on track & working on it again
Reporter | ||
Comment 6•5 years ago
|
||
Thanks Vaibhav!
Reporter | ||
Updated•5 years ago
|
Reporter | ||
Comment 7•5 years ago
|
||
Reporter | ||
Updated•5 years ago
|
Reporter | ||
Comment 8•5 years ago
|
||
Reporter | ||
Comment 9•5 years ago
|
||
Released in generic-worker 16.2.0.
Reporter | ||
Comment 10•5 years ago
•
|
||
Note: testing on gecko Windows worker types in this try push.
Reporter | ||
Comment 11•5 years ago
|
||
Updated•5 years ago
|
Description
•