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

Exp: Using native CORS mw lib #2267

Closed
wants to merge 6 commits into from
Closed

Exp: Using native CORS mw lib #2267

wants to merge 6 commits into from

Conversation

RobinTail
Copy link
Owner

@RobinTail RobinTail commented Dec 25, 2024

@RobinTail RobinTail added refactoring The better way to achieve the same result breaking Backward incompatible changes labels Dec 25, 2024
@RobinTail RobinTail added this to the v22 milestone Dec 25, 2024
Copy link

coveralls-official bot commented Dec 25, 2024

Coverage Status

coverage: 99.886% (-0.1%) from 100.0%
when pulling 2152036 on use-cors
into 5462dd6 on master.

@RobinTail RobinTail mentioned this pull request Jan 1, 2025
@RobinTail
Copy link
Owner Author

I don't like how that package cors is maintained.
It has substantial amount of issues that are not being solved for 6 years.
Therefore, I think, at least at the moment, I rather will keep the CORS implementation open for user customization.

@RobinTail RobinTail closed this Jan 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking Backward incompatible changes refactoring The better way to achieve the same result
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant