apache / apache/rocketmq

[Bug] Proxy remoting CHECK_CLIENT_CONFIG always returns success

Open
#10,795 2 comments 0 reactions 0 assignees View on GitHub
Dominant language
Java
Stars
22.6k
Forks
12k
Avg merge
3d 1h
Merged PRs (30d)
27

Description

### What happened?

The Proxy remoting implementation of `RequestCode.CHECK_CLIENT_CONFIG` currently creates a success response without validating the request body.

Broker-side handling validates `CheckClientRequestBody`, including filter expression type support and expression parse errors, and returns an error response when the client config is invalid. The Proxy remoting path should preserve that behavior instead of always returning success.

### Expected behavior

Proxy remoting `CHECK_CLIENT_CONFIG` should decode and validate the request in the same way as the broker path, returning an error response for unsupported or invalid subscription expressions.

### Scope

RocketMQ Proxy / remoting compatibility. This is related to the Proxy Admin and client diagnostics track because invalid client subscription configuration should be visible through the Proxy path.

### Suggested fix

Decode `CheckClientRequestBody`, validate expression type and expression syntax, and return the corresponding remoting error code when validation fails.

Contributor guide

Open the contributing guide

Research direction

Trace the Proxy remoting handler for RequestCode.CHECK_CLIENT_CONFIG and compare it with the broker-side handling of CheckClientRequestBody. Check how expression type support, parse errors, and remoting error codes are handled; done means invalid subscription expressions produce an error response through Proxy remoting instead of success.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
api, backend
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
55/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.