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.

Sign up for GitHub

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

Jump to bottom

[Static web assets] Be more leanient when applying compression negotiation #45598

Draft
wants to merge 3 commits into
base: release/9.0.1xx
Choose a base branch
from
Draft

[Static web assets] Be more leanient when applying compression negotiation #45598

wants to merge 3 commits into from

Conversation

javiercn
Copy link
Member

@javiercn javiercn commented Dec 20, 2024 •
edited
Loading

@dotnet-issue-labeler dotnet-issue-labeler bot added Area-AspNetCore RazorSDK, BlazorWebAssemblySDK, dotnet-watch untriaged Request triage from a team member labels Dec 20, 2024
@dotnet-policy-service Dotnet Policy Service
Copy link
Contributor

Thanks for your PR, @javiercn.
To learn about the PR process and branching schedule of this repo, please take a look at the SDK PR Guide.

@javiercn javiercn force-pushed the javiercn/compression-fixes branch from 8d5c6d1 to 2e40457 Compare December 20, 2024 20:45
This was referenced Dec 20, 2024
Open
Open
Open
Open
@javiercn javiercn force-pushed the javiercn/compression-fixes branch from 78b60f2 to 261c4e4 Compare January 7, 2025 10:57
@lofcz lofcz mentioned this pull request Jan 14, 2025
Open
@javiercn javiercn force-pushed the javiercn/compression-fixes branch 2 times, most recently from 05af09f to 86b45e0 Compare February 5, 2025 12:10
@javiercn javiercn changed the base branch from main to release/9.0.1xx February 5, 2025 12:11
@javiercn javiercn force-pushed the javiercn/compression-fixes branch from 86b45e0 to 997abfb Compare February 5, 2025 12:12
@lofcz
Copy link

lofcz commented Feb 16, 2025

Is there an estimate of when this could be reviewed and merged? This solves a critical issue affecting multiple users reported in at least two issues: dotnet/aspnetcore#59625 dotnet/aspnetcore#59476

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

Copilot code review Copilot Awaiting requested review from Copilot Copilot will automatically review once the pull request is marked ready for review

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
Area-AspNetCore RazorSDK, BlazorWebAssemblySDK, dotnet-watch Branch Lockdown untriaged Request triage from a team member
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@javiercn @lofcz