Skip to content

Conversation

@danielsantosdfs13
Copy link

@danielsantosdfs13 danielsantosdfs13 commented Jul 3, 2025

Fix this issue: #3879

Checklist:

  • Either (a) I've created an enhancement proposal and discussed it with the community, (b) this is a bug fix, or (c) this is a chore.
  • The title of the PR is (a) conventional with a list of types and scopes found here, (b) states what changed, and (c) suffixes the related issues number. E.g. "fix(controller): Updates such and such. Fixes #1234".
  • I've signed my commits with DCO
  • I have written unit and/or e2e tests for my change. PRs without these are unlikely to be merged.
  • My builds are green. Try syncing with master if they are not.
  • My organization is added to USERS.md.

@danielsantosdfs13 danielsantosdfs13 force-pushed the fix/3879 branch 3 times, most recently from 599d3a9 to 1a5b445 Compare July 3, 2025 13:38
@github-actions
Copy link
Contributor

github-actions bot commented Jul 3, 2025

Published E2E Test Results

4 files  4 suites   0s ⏱️
1 tests 0 ✅ 0 💤 1 ❌
0 runs  -4 ✅ 0 💤 4 ❌

For more details on these failures, see this check.

Results for commit 2dcd5d2.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Contributor

github-actions bot commented Jul 3, 2025

Published Unit Test Results

1 files  129 suites   0s ⏱️
6 tests   5 ✅ 0 💤   1 ❌
5 runs  -113 ✅ 0 💤 118 ❌

For more details on these failures, see this check.

Results for commit 2dcd5d2.

♻️ This comment has been updated with latest results.

@zachaller zachaller added this to the v1.9 milestone Jul 3, 2025
Signed-off-by: Daniel Santos <[email protected]>
…atchIncludes the implementation of deep copies for the Method and Path fields in the HeaderRoutingMatch structure, ensuring that references are correctly duplicated during the DeepCopy operation.

Signed-off-by: Daniel Santos <[email protected]>
Signed-off-by: Daniel Santos <[email protected]>
@sonarqubecloud
Copy link

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.

2 participants