Skip to content

fix: CORS defaults#4718

Open
pablocarle wants to merge 32 commits into
v2.x.xfrom
reboot/fix/permissive_cors
Open

fix: CORS defaults#4718
pablocarle wants to merge 32 commits into
v2.x.xfrom
reboot/fix/permissive_cors

Conversation

@pablocarle

@pablocarle pablocarle commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Description

  • Remove support for CORS customization in Spring Cloud Gateway in version 2 Zowe.
  • Remove default to * in Zuul Gateway. It now defaults to a provided list of URLs via configuration property OR to the gateway's hostname + port if not provided.
  • Added integration tests for these scenarios

Type of change

  • fix: Bug fix (non-breaking change which fixes an issue)

Checklist:

  • My code follows the style guidelines of this project
  • PR title conforms to commit message guideline ## Commit Message Structure Guideline
  • I have commented my code, particularly in hard-to-understand areas. In JS I did provide JSDoc
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • The java tests in the area I was working on leverage @nested annotations
  • Any dependent changes have been merged and published in downstream modules

richard-salac and others added 7 commits June 10, 2026 13:03
Signed-off-by: Richard Salac <richard.salac@broadcom.com>
Signed-off-by: Pablo Carle <pablo.carle@broadcom.com>
Signed-off-by: Pablo Carle <pablo.carle@broadcom.com>
wip
Signed-off-by: Pablo Carle <pablo.carle@broadcom.com>
wip
Signed-off-by: Pablo Carle <pablo.carle@broadcom.com>
Pablo Carle added 2 commits June 15, 2026 16:54
Signed-off-by: Pablo Carle <pablo.carle@broadcom.com>
Signed-off-by: Pablo Carle <pablo.carle@broadcom.com>
Pablo Carle added 2 commits June 16, 2026 15:27
Signed-off-by: Pablo Carle <pablo.carle@broadcom.com>
Signed-off-by: Pablo Carle <pablo.carle@broadcom.com>
@EvaJavornicka EvaJavornicka moved this from New to In Progress in API Mediation Layer Backlog Management Jun 17, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Ported from v3 to help fix issues in acceptance tests

@pablocarle pablocarle changed the title Reboot/fix/permissive cors fix: CORS defaults Jun 18, 2026
pablocarle and others added 5 commits June 18, 2026 16:06
Signed-off-by: Pablo Carle <pablo.carle@broadcom.com>
Signed-off-by: Pablo Carle <pablo.carle@broadcom.com>
Signed-off-by: Pablo Carle <pablo.carle@broadcom.com>
Signed-off-by: Pablo Carle <pablo.carle@broadcom.com>
@pablocarle pablocarle marked this pull request as ready for review June 19, 2026 15:09
Pablo Carle and others added 12 commits June 19, 2026 17:29
Signed-off-by: Pablo Carle <pablo.carle@broadcom.com>
Signed-off-by: Pablo Carle <pablo.carle@broadcom.com>
Signed-off-by: Pablo Carle <pablo.carle@broadcom.com>
Signed-off-by: Pablo Carle <pablo.carle@broadcom.com>
Signed-off-by: Pablo Carle <pablo.carle@broadcom.com>
Signed-off-by: Pablo Carle <pablo.carle@broadcom.com>
Signed-off-by: Pablo Carle <pablo.carle@broadcom.com>
Signed-off-by: Pablo Carle <pablo.carle@broadcom.com>
@sonarqubecloud

Copy link
Copy Markdown

@pablocarle pablocarle mentioned this pull request Jun 26, 2026
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Sensitive Sensitive change that requires peer review size/XXL

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

3 participants