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

fix: Do not require auth on CORS preflights #283

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

berezovskyi
Copy link
Contributor

@berezovskyi berezovskyi commented Oct 5, 2024

Description

Do not require auth on CORS preflights. Also allows the ignore flag to be overridden too.

Checklist

  • This PR adds an entry to the CHANGELOG. _See https://keepachangelog.com/en/1.0.0/ for instructions. Minor edits are exempt.
  • This PR expects some manual changes to the generated code. The needed changes are detailed in the CHANGELOG entry.
  • This PR was tested with at least one code generation, resulting in a working OSLC server.
  • This PR does NOT break the user block code

Issues

Fixes #214

(use exactly this syntax to link to issues, one issue per statement only!)

Also allows the ignore flag to be overridden too
@berezovskyi
Copy link
Contributor Author

And I think your recent regen for 6.0 wiped out oslc-op/refimpl@3500741 and maybe some other changes.

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.

OAuth blocks CORS preflight requests
1 participant