Skip to content

refactor(index): change default methods to cors-safelisted methods#359

Merged
climba03003 merged 3 commits intomainfrom
refactor/safelist
Feb 27, 2025
Merged

refactor(index): change default methods to cors-safelisted methods#359
climba03003 merged 3 commits intomainfrom
refactor/safelist

Conversation

@Fdawgs
Copy link
Member

@Fdawgs Fdawgs commented Feb 18, 2025

Closes #358.

This is a breaking change and will require a semver major release.

Checklist

@Fdawgs Fdawgs requested a review from a team February 19, 2025 09:01
@Fdawgs Fdawgs requested a review from climba03003 February 19, 2025 15:20
Copy link
Member

@climba03003 climba03003 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but please wait for one more approval before merging.

@Fdawgs Fdawgs requested a review from a team February 24, 2025 06:02
Signed-off-by: Frazer Smith <frazer.dev@icloud.com>
@Fdawgs Fdawgs requested a review from jean-michelet February 26, 2025 21:33
Copy link
Member

@gurgunday gurgunday left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@climba03003 climba03003 merged commit 0d7b9fa into main Feb 27, 2025
13 checks passed
@climba03003 climba03003 deleted the refactor/safelist branch February 27, 2025 06:01
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.

Change default for methods option value to safe values

4 participants