X Tutup
Skip to content

fix: this allows to implement OPTIONS request handling in framework c…#38758

Merged
DeepDiver1975 merged 2 commits intomasterfrom
fix/allow-options-requests-to-be-handled-by-controllers
Dec 1, 2021
Merged

fix: this allows to implement OPTIONS request handling in framework c…#38758
DeepDiver1975 merged 2 commits intomasterfrom
fix/allow-options-requests-to-be-handled-by-controllers

Conversation

@DeepDiver1975
Copy link
Member

…ontrollers - if it is not a CORS preflight request. CORS requests will continue to be handled by core

Description

Related Issue

  • Fixes <issue_link>

Motivation and Context

How Has This Been Tested?

  • test environment:
  • test case 1:
  • test case 2:
  • ...

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Database schema changes (next release will require increase of minor version instead of patch)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised:
  • Changelog item, see TEMPLATE

@update-docs
Copy link

update-docs bot commented May 20, 2021

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@AlexAndBear
Copy link

@JammingBen fyi

@JammingBen
Copy link
Contributor

@DeepDiver1975 Is this still needed?

@DeepDiver1975
Copy link
Member Author

@DeepDiver1975 Is this still needed?

Don't needed anymore for my original case ..... but worth it as a general enhancement ...

DeepDiver1975 and others added 2 commits September 17, 2021 08:08
…ontrollers - if it is not a CORS preflight request. CORS requests will continue to be handled by core
@JammingBen JammingBen force-pushed the fix/allow-options-requests-to-be-handled-by-controllers branch from f4a08c2 to 3a3e580 Compare September 17, 2021 06:10
@JammingBen
Copy link
Contributor

I rebased and added a changelog item. @DeepDiver1975 If you're fine with it we can merge.

@JammingBen JammingBen marked this pull request as ready for review September 17, 2021 06:11
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@DeepDiver1975 DeepDiver1975 merged commit 6df2c36 into master Dec 1, 2021
@delete-merged-branch delete-merged-branch bot deleted the fix/allow-options-requests-to-be-handled-by-controllers branch December 1, 2021 09:51
@phil-davis
Copy link
Contributor

@DeepDiver1975 this is not in the release-10.9.0 branch.
Does it need to be released "soon" in 10.9.0?

@DeepDiver1975
Copy link
Member Author

Does it need to be released "soon" in 10.9.0?

no - thx

IljaN pushed a commit that referenced this pull request Dec 6, 2021
#38758)

* fix: this allows to implement OPTIONS request handling in framework controllers - if it is  not a CORS preflight request. CORS requests will continue to be handled by core

* Add changelog item for #38758

Co-authored-by: Jannik Stehle <jannik.stehle@gmail.com>
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.

4 participants

X Tutup