`release::releases::get_logs()` returns unprintable/garbled string
Nobody has claimed this yet.
Assessment
- Difficulty
- 5/5
- Estimated time
- Over a week
- Newbie friendliness
- 25/100
Research direction
Start with release::releases::get_logs() and inspect its response headers, especially content-type. Compare other API functions that may return compressed data, then decide whether to decompress or expose Result<Vec>; done when the return contract and documentation consistently reflect the chosen behavior.
Written by the indexing model from the issue text.
Description
release::releases::get_logs() is currently defined to return Result<String>. However, it currently returns a lot of unprintable/garbled data...
Inspection of the response headers shows that the data is in compressed format:
content-type: application/zip; api-version=7.1-preview.2
Need to decide whether this API function should:
- Auto-decompress the data, by inspecting the
content-typeheader and doing unzip before returning the data asResult<String> - Document the fact that the returned data is zipped, and change the return type to be raw data:
Result<Vec<u8>>, leaving it up to the application to decompress (if required - app might just want to write the logs as a compressed file...)
I expect that this might apply to other API functions too.
- Dominant language
- Rust
- Stars
- 76
- Forks
- 27
- Avg merge
- 6d 8h
- Merged PRs (30d)
- 2
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from microsoft/azure-devops-rust-api
-
Difficulty 5/5 Over a week Newbie friendliness 30/100
-
Difficulty 4/5 3-5 days Newbie friendliness 42/100
microsoft/azure-devops-rust-api#590 · 2 comments ·
-
Difficulty 5/5 Over a week Newbie friendliness 20/100
microsoft/azure-devops-rust-api#552 · 1 reaction ·
-
enhancement
Difficulty 5/5 Over a week Newbie friendliness 30/100
microsoft/azure-devops-rust-api#309 · 1 comment · 2 reactions ·
-
Difficulty 3/5 1-2 days Newbie friendliness 42/100
microsoft/azure-devops-rust-api#181 · 4 comments ·
All issues in microsoft/azure-devops-rust-api
Similar issues
-
bug github_actions
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
registrystack/registry-stack#1393 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
longbridge/gpui-kit#3223 ·
-
bug engine
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
rocky-data/rocky#2181 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
oasisprotocol/oasis-sdk#2523 ·
-
[indexer] [QA] Add a focused test for the new NonRetryableError / assertSocketAlive() behavior. Openbot:ai-assisted component:indexer QA-roadmap status:untriaged
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
midnightntwrk/midnight-indexer#1557 ·