Skip to content

Fixes #6484: Guard missing Dubbo ingress endpoints - #7220

Open
BobSong-dev wants to merge 2 commits into
apache:masterfrom
BobSong-dev:fix/6484-dubbo-ingress-endpoints
Open

BobSong-dev wants to merge 2 commits into
apache:masterfrom
BobSong-dev:fix/6484-dubbo-ingress-endpoints

Conversation

@BobSong-dev

@BobSong-dev BobSong-dev commented Sep 24, 2026 •

Copy link
Copy Markdown
Contributor

Fixes #6484

Background

DubboIngressParser dereferences the Endpoints informer result before checking it. During ingress reconciliation, the referenced Service may exist before its Endpoints resource is available, causing parsing of a default or path backend to throw a NullPointerException.

Changes

  • Return an empty upstream list and log when Endpoints are unavailable for a Dubbo default backend;
  • Return an empty upstream list and log when Endpoints are unavailable for a Dubbo path backend;
  • Add regression coverage for both missing-Endpoints paths.

Verification

  • ./mvnw test -pl shenyu-kubernetes-controller -Dtest=DubboIngressParserTest -Dmaven.javadoc.skip=true -Drat.skip=true -Djacoco.skip=true: BUILD SUCCESS; 9 tests passed;
  • git diff --check: passed;

This branch has not been deployed

No deployments
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.

[BUG] Dubbo ingress parsing throws NPE when endpoints are missing

1 participant