Skip to content

fix: Use relaxed firewall only for routed request (v2)#4666

Merged
balhar-jakub merged 1 commit into
v2.x.xfrom
reboot/relaxed-firewall-only-for-gw-v2
Jun 3, 2026
Merged

fix: Use relaxed firewall only for routed request (v2)#4666
balhar-jakub merged 1 commit into
v2.x.xfrom
reboot/relaxed-firewall-only-for-gw-v2

Conversation

@pavel-jares-bcm
Copy link
Copy Markdown
Contributor

Description

This PR is an equivalent of #4643 for v2:

Currently the whole services uses a relaxed firewalls. It is based on an original issue #23. Some services requires to provide these type of URLs. Anyway, it is valid only for gateway - the routing part. For the rest of the endpoints there could be use a strict firewall. This PR allows to define relaxed one only for proxying.

Linked to # (issue)
Part of the # (epic)

Type of change

Please delete options that are not relevant.

  • fix: Bug fix (non-breaking change which fixes an issue)
  • feat: New feature (non-breaking change which adds functionality)
  • docs: Change in a documentation
  • refactor: Refactor the code
  • chore: Chore, repository cleanup, updates the dependencies.
  • BREAKING CHANGE or !: Breaking change (fix or feature that would cause existing functionality to not work as expected)

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

For more details about how should the code look like read the Contributing guideline

Signed-off-by: Pavel Jareš <Pavel.Jares@broadcom.com>
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Jun 2, 2026

@pavel-jares-bcm pavel-jares-bcm marked this pull request as ready for review June 2, 2026 15:11
@balhar-jakub balhar-jakub merged commit 08cfc87 into v2.x.x Jun 3, 2026
58 of 60 checks passed
@balhar-jakub balhar-jakub deleted the reboot/relaxed-firewall-only-for-gw-v2 branch June 3, 2026 06:13
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/L

Projects

Development

Successfully merging this pull request may close these issues.

2 participants