Skip to content

[v0.14] Fix GitRepo URL matching in webhook server#4779

Merged
weyfonk merged 1 commit into
rancher:release/v0.14from
weyfonk:4648-0.14-webhook-gitrepo-url-match
Mar 10, 2026
Merged

[v0.14] Fix GitRepo URL matching in webhook server#4779
weyfonk merged 1 commit into
rancher:release/v0.14from
weyfonk:4648-0.14-webhook-gitrepo-url-match

Conversation

@weyfonk
Copy link
Copy Markdown
Contributor

@weyfonk weyfonk commented Mar 10, 2026

The webhook server would previously erroneously match an incoming request's repository URL with a GitRepo whose spec repo amounted to that URL with a suffix.
The regular expression for repository URL matching no longer tolerates suffixes.

Refers to #4648.
Backport of #4777.

Additional Information

Checklist

- [ ] I have updated the documentation via a pull request in the
fleet-docs repository.

The webhook server would previously erroneously match an incoming
request's repository URL with a GitRepo whose spec repo amounted to that
URL with a suffix.
The regular expression for repository URL matching no longer tolerates
suffixes.
@weyfonk weyfonk requested a review from a team as a code owner March 10, 2026 08:02
@weyfonk weyfonk self-assigned this Mar 10, 2026
@weyfonk weyfonk added this to Fleet Mar 10, 2026
@weyfonk weyfonk moved this to 👀 In review in Fleet Mar 10, 2026
@weyfonk weyfonk added this to the v2.13.5 milestone Mar 10, 2026
@weyfonk weyfonk merged commit 69a250b into rancher:release/v0.14 Mar 10, 2026
34 of 37 checks passed
@github-project-automation github-project-automation Bot moved this from 👀 In review to ✅ Done in Fleet Mar 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants