Skip to content

fix: decompress Gzipped data #317

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 5 commits into from
May 6, 2022
Merged

fix: decompress Gzipped data #317

merged 5 commits into from
May 6, 2022

Conversation

bednar
Copy link
Contributor

@bednar bednar commented Apr 28, 2022

Closes #310

Proposed Changes

  1. Fixed decompressing Gzipped data
  2. Updated JetBrains.ReSharper.GlobalTools (code formatter) to 2022.1.0 3.
  3. Fixed logging HTTP headers from streaming request

Checklist

  • CHANGELOG.md updated
  • Rebased/mergeable
  • A test has been added if appropriate
  • dotnet test completes successfully
  • Commit messages are in semantic format
  • Sign CLA (if not already signed)

@codecov-commenter
Copy link

Codecov Report

Merging #317 (ab30bba) into master (cccc410) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #317      +/-   ##
==========================================
- Coverage   88.13%   88.13%   -0.01%     
==========================================
  Files          76       76              
  Lines        6526     6531       +5     
==========================================
+ Hits         5752     5756       +4     
- Misses        774      775       +1     
Impacted Files Coverage Δ
Client.Core/Internal/AbstractQueryClient.cs 84.29% <100.00%> (+0.33%) ⬆️
Client/Internal/RetryAttempt.cs 99.02% <0.00%> (-0.98%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cccc410...ab30bba. Read the comment docs.

@bednar bednar requested a review from rhajek April 28, 2022 08:58
@bednar bednar marked this pull request as ready for review April 28, 2022 08:58
Copy link
Contributor

@rhajek rhajek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@bednar bednar merged commit bf40ba6 into master May 6, 2022
@bednar bednar deleted the gzip branch May 6, 2022 08:54
@bednar bednar added this to the 4.2.0 milestone May 6, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

queryApi.QueryAsync returns GZipped request
3 participants