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

v5.x DRAFT: CORS rewrite #226

Merged
merged 33 commits into from
Mar 25, 2024
Merged

v5.x DRAFT: CORS rewrite #226

merged 33 commits into from
Mar 25, 2024

Conversation

kwhitley
Copy link
Owner

Description

Staging changes to createCors... more to follow.

Type of Change (select one and follow subtasks)

  • Documentation (README.md)
  • Maintenance or repo-level work (e.g. linting, build, tests, refactoring, etc.)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
    • I have included test coverage
  • Breaking change (fix or feature that would cause existing functionality/userland code to not work as expected)
    • Explain why a breaking change is necessary:
  • This change requires (or is) a documentation update
    • I have added necessary local documentation (if appropriate)
    • I have added necessary itty.dev documentation (if appropriate)

kwhitley added 18 commits March 14, 2024 13:33
@kwhitley
flowrouter is the new router and ittyrouter is the old router
45fea10
@kwhitley
further evolving the standard
bd8ee7b
@kwhitley
added shared router tests for feature parity between IttyRouter and R…
4d8183c 
…otuer
@kwhitley
take that, linter!
502ff7b
@kwhitley
cleaned up example
a3b471b
@kwhitley
added AutoRouter test coverage
edeebfa
@kwhitley
lint fix
9d02b27
@kwhitley
types for before/after/onError
b31ddf4
@kwhitley
released v4.3.0-next.0 - releasing v4.3 as next to determine sizes
3ac4cdc
@kwhitley
Update README.md
2db8606
@kwhitley
Update README.md
bc7c7e6
@kwhitley
Update README.md
9be0a6a
@kwhitley
Update README.md
2880450
@kwhitley
Update README.md
65f4ad1
@kwhitley
catch is a single handler
9a70da3
@kwhitley
Merge branch 'auto-router-v2' of github.com:kwhitley/itty-router into…
2e39069 
… auto-router-v2
@kwhitley
v4.3 DRAFT: rename after stage to finally (#225)
201cdce 
* replaced after stage with finally

* fixed potential bug in createResponse to prevent pollution if passed a Request as second param, like in v4.3 stages

* createResponse simplified and safe against using as ResponseHandler
@kwhitley
staging changes for cors revamp
a11ea47
@kwhitley kwhitley added BUG Something isn't working new feature New feature or request in progress I'm working on it... allegedly... labels Mar 22, 2024
@kwhitley kwhitley self-assigned this Mar 22, 2024
@kwhitley kwhitley changed the title DRAFT: v4.3 CORS revamp v4.3 DRAFT: CORS revamp Mar 22, 2024
@kwhitley kwhitley marked this pull request as draft March 22, 2024 17:42
@kwhitley kwhitley changed the title v4.3 DRAFT: CORS revamp v5.x DRAFT: CORS rewrite Mar 24, 2024
@kwhitley kwhitley changed the base branch from auto-router-v2 to v5.x March 24, 2024 20:11
@kwhitley kwhitley removed the BUG Something isn't working label Mar 25, 2024
@kwhitley kwhitley mentioned this pull request Mar 25, 2024
Closed
1 task
This was referenced Mar 25, 2024
Closed
Closed
Closed
@kwhitley kwhitley marked this pull request as ready for review March 25, 2024 22:39
@kwhitley kwhitley merged commit 8357b10 into v5.x Mar 25, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@kwhitley kwhitley

Labels
in progress I'm working on it... allegedly... new feature New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@kwhitley