Skip to content

Commit f7df43c

Browse files
committed
Remove TraceInterceptor support from AutoJsonRpcServiceImplExporter and mark access methods as deprected
RemoteExporter is deprecated and has been removed from Spring Framework starting from v6. This change removes registerTraceInterceptor parameter registration in RemoteExporter. Methods are marked as @deprecated to preserve compile-time compatibility. Signed-off-by: cyb3r4nt <104218001+cyb3r4nt@users.noreply.github.com>
1 parent 2687b29 commit f7df43c

File tree

2 files changed

+14
-16
lines changed

2 files changed

+14
-16
lines changed

src/main/java/com/googlecode/jsonrpc4j/spring/AutoJsonRpcServiceExporter.java

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,6 @@ public class AutoJsonRpcServiceExporter implements BeanFactoryPostProcessor {
4444

4545
private ObjectMapper objectMapper;
4646
private ErrorResolver errorResolver = null;
47-
private Boolean registerTraceInterceptor;
4847
private boolean backwardsCompatible = true;
4948
private boolean rethrowExceptions = false;
5049
private boolean allowExtraParams = false;
@@ -141,10 +140,6 @@ private void registerServiceProxy(DefaultListableBeanFactory defaultListableBean
141140
builder.addPropertyValue("invocationListener", invocationListener);
142141
}
143142

144-
if (registerTraceInterceptor != null) {
145-
builder.addPropertyValue("registerTraceInterceptor", registerTraceInterceptor);
146-
}
147-
148143
if (httpStatusCodeProvider != null) {
149144
builder.addPropertyValue("httpStatusCodeProvider", httpStatusCodeProvider);
150145
}
@@ -225,12 +220,16 @@ public void setAllowLessParams(boolean allowLessParams) {
225220
}
226221

227222
/**
228-
* See {@link org.springframework.remoting.support.RemoteExporter#setRegisterTraceInterceptor(boolean)}
223+
* See {@code org.springframework.remoting.support.RemoteExporter#setRegisterTraceInterceptor(boolean)}
224+
* <p>
225+
* Note: this method is deprecated and marked for removal.
226+
* {@code RemoteExporter} and {@code TraceInterceptor-s} are no longer supported.
229227
*
230228
* @param registerTraceInterceptor the registerTraceInterceptor value to set
231229
*/
230+
@Deprecated
232231
public void setRegisterTraceInterceptor(boolean registerTraceInterceptor) {
233-
this.registerTraceInterceptor = registerTraceInterceptor;
232+
// NOOP
234233
}
235234

236235
/**

src/main/java/com/googlecode/jsonrpc4j/spring/AutoJsonRpcServiceImplExporter.java

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,6 @@ public class AutoJsonRpcServiceImplExporter implements BeanFactoryPostProcessor
5050

5151
private ObjectMapper objectMapper;
5252
private ErrorResolver errorResolver = null;
53-
private Boolean registerTraceInterceptor;
5453
private boolean backwardsCompatible = true;
5554
private boolean rethrowExceptions = false;
5655
private boolean allowExtraParams = false;
@@ -181,11 +180,7 @@ private void registerServiceProxy(DefaultListableBeanFactory defaultListableBean
181180
if (invocationListener != null) {
182181
builder.addPropertyValue("invocationListener", invocationListener);
183182
}
184-
185-
if (registerTraceInterceptor != null) {
186-
builder.addPropertyValue("registerTraceInterceptor", registerTraceInterceptor);
187-
}
188-
183+
189184
if (httpStatusCodeProvider != null) {
190185
builder.addPropertyValue("httpStatusCodeProvider", httpStatusCodeProvider);
191186
}
@@ -280,14 +275,18 @@ public void setAllowExtraParams(boolean allowExtraParams) {
280275
public void setAllowLessParams(boolean allowLessParams) {
281276
this.allowLessParams = allowLessParams;
282277
}
283-
278+
284279
/**
285-
* See {@link org.springframework.remoting.support.RemoteExporter#setRegisterTraceInterceptor(boolean)}
280+
* See {@code org.springframework.remoting.support.RemoteExporter#setRegisterTraceInterceptor(boolean)}
281+
* <p>
282+
* Note: this method is deprecated and marked for removal.
283+
* {@code RemoteExporter} and {@code TraceInterceptor-s} are no longer supported.
286284
*
287285
* @param registerTraceInterceptor the registerTraceInterceptor value to set
288286
*/
287+
@Deprecated
289288
public void setRegisterTraceInterceptor(boolean registerTraceInterceptor) {
290-
this.registerTraceInterceptor = registerTraceInterceptor;
289+
// NOOP
291290
}
292291

293292
/**

0 commit comments

Comments
 (0)