-
Notifications
You must be signed in to change notification settings - Fork 9.8k
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
Add AnyHeader and AnyMethod to src/Middleware/CORS/src/Infrastructure/CorsConstants.cs #54772
Labels
api-approved
API was approved in API review, it can be implemented
area-networking
Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions
Comments
hamidrezahy
added
the
api-suggestion
Early API idea and discussion, it is NOT ready for implementation
label
Mar 26, 2024
dotnet-issue-labeler
bot
added
the
area-networking
Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions
label
Mar 26, 2024
amcasey
added
api-ready-for-review
API is ready for formal API review - https://github.com/dotnet/apireviews
and removed
api-suggestion
Early API idea and discussion, it is NOT ready for implementation
labels
Mar 26, 2024
Thank you for submitting this for API review. This will be reviewed by @dotnet/aspnet-api-review at the next meeting of the ASP.NET Core API Review group. Please ensure you take a look at the API review process documentation and ensure that:
|
API Approved! |
amcasey
added
api-approved
API was approved in API review, it can be implemented
and removed
api-ready-for-review
API is ready for formal API review - https://github.com/dotnet/apireviews
labels
Mar 28, 2024
4 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
api-approved
API was approved in API review, it can be implemented
area-networking
Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions
Background and Motivation
According to pull request #54247, CorsConstants only contain AnyOrigin value
This is while it also needs AnyHeaders and AnyMethods because, unlike AnyOrigin, the value "*" is used directly in other places.
Proposed API
My changes are like this:
Usage Examples
Instead of using this form:
The text was updated successfully, but these errors were encountered: