Skip to content

fix: dont allow paths in grpc protocol#3309

Open
rakesh-garimella wants to merge 1 commit intokyma-project:mainfrom
rakesh-garimella:fix-grpc-check
Open

fix: dont allow paths in grpc protocol#3309
rakesh-garimella wants to merge 1 commit intokyma-project:mainfrom
rakesh-garimella:fix-grpc-check

Conversation

@rakesh-garimella
Copy link
Copy Markdown
Contributor

Description

Changes proposed in this pull request (what was done and why):

  • grpc protocol does not support path
  • When we set the path along with port its not validated. eg. https://foo.bar:4317/my/path was accepted and caused the otel collector to crashloop.

Changes refer to particular issues, PRs or documents:

Traceability

  • The PR is linked to a GitHub issue.
  • The follow-up issues (if any) are linked in the Related Issues section.
  • If the change is user-facing, the documentation has been adjusted.
  • If a CRD is changed, the corresponding Busola ConfigMap has been adjusted.
  • The feature is unit-tested.
  • The feature is e2e-tested.

@rakesh-garimella rakesh-garimella requested a review from a team as a code owner March 27, 2026 10:10
@github-actions github-actions bot added this to the 1.61.0 milestone Mar 27, 2026
@github-actions github-actions bot added the kind/bug Categorizes issue or PR as related to a bug. label Mar 27, 2026
@rakesh-garimella rakesh-garimella added the area/manager Manager or module changes label Mar 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/manager Manager or module changes kind/bug Categorizes issue or PR as related to a bug.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant