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

feat: Implement global CORS #458

Merged
merged 5 commits into from
Apr 9, 2023
Merged

feat: Implement global CORS #458

merged 5 commits into from
Apr 9, 2023

Conversation

sansyrox
Copy link
Member

@sansyrox sansyrox commented Apr 2, 2023

Description

This PR partially implements CORS.

I would also like to implement per-route CORS, However, before implementing per route CORS, I would like to implement middleware chaining. This will substantially improve the way we structure our code.

@ghost
Copy link

ghost commented Apr 2, 2023

👇 Click on the image for a new way to code review

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map legend

@codspeed-hq
Copy link

codspeed-hq bot commented Apr 2, 2023

CodSpeed Performance Report

Merging #458 feat/cors (0712e43) will not alter performances.

Summary

🔥 0 improvements
❌ 0 regressions
✅ 74 untouched benchmarks

🆕 1 new benchmarks
⁉️ 0 dropped benchmarks

Benchmarks breakdown

Benchmark main feat/cors Change
🆕 test_allow_cors N/A 846 µs N/A

@sansyrox sansyrox force-pushed the main branch 4 times, most recently from f0bf73a to 97e2c0c Compare April 5, 2023 19:13
@sansyrox sansyrox marked this pull request as ready for review April 7, 2023 06:58
sansyrox and others added 2 commits April 9, 2023 05:05
Before implementing per route CORS, I would like to implement middleware
chaining.
@sansyrox sansyrox force-pushed the feat/cors branch 4 times, most recently from 930ca20 to f860a52 Compare April 9, 2023 04:11
@sansyrox sansyrox force-pushed the feat/cors branch 3 times, most recently from bb6da24 to 3491ee9 Compare April 9, 2023 04:45
@sansyrox sansyrox merged commit 4083bfc into main Apr 9, 2023
@sansyrox sansyrox deleted the feat/cors branch April 9, 2023 04:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant