Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

graph: Allow gzip compression in requests #3278

Merged
merged 1 commit into from
Nov 27, 2023

Conversation

mbarnes
Copy link
Contributor

@mbarnes mbarnes commented Nov 10, 2023

Microsoft fixed the Graph service regression that broke kiota-http-go, so revert the workaround introduced in #3269.

Copy link
Collaborator

@SudoBrendan SudoBrendan left a comment

Choose a reason for hiding this comment

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

Thanks for coming back with the cleanup!

@cadenmarchese
Copy link
Collaborator

@mbarnes mind doing a rebase on this branch? The failing npm-audit-portal-v2 test was fixed, but the check is required for merge.

Microsoft fixed the Graph service regression that broke
kiota-http-go, so revert the workaround.
@mbarnes mbarnes added next-release To be included in the next RP release rollout and removed ready-for-review labels Nov 22, 2023
@cadenmarchese cadenmarchese merged commit 966f7a1 into Azure:master Nov 27, 2023
18 checks passed
@mbarnes mbarnes deleted the msgraph-gzip-maybe-fixed branch November 27, 2023 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
loki next-release To be included in the next RP release rollout size-small Size small
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants