[INLONG-9] Remove some unnecessary code

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 0.3.0
• Labels: pull-request-available
• Assignee: lan.liang
• Reporter: lan.liang
• Created: 07/Jan/20 15:56
• Updated: 28/Jul/21 11:13
• Resolved: 13/Jan/20 02:32

Description

1. 'String.valueOf()'

String strCallbackFun = req.getParameter("callback");
if ((TStringUtils.isNotEmpty(strCallbackFun))
        && (strCallbackFun.length() <= TBaseConstants.META_MAX_CALLBACK_STRING_LENGTH)
        && (strCallbackFun.matches(TBaseConstants.META_TMP_CALLBACK_STRING_VALUE))) {
    strCallbackFun = String.valueOf(strCallbackFun).trim();
}

 

com.tencent.tubemq.server.master.web.action.screen.Webapi#execute

 

2. {String}.substring()

if (path.startsWith("/")) {
    path = path.substring(1, path.length());
}

com.tencent.tubemq.server.master.web.simplemvc.RequestContext#normalizePath

 

3.unuse init boolean

private boolean isOverTLS = false;

com.tencent.tubemq.corerpc.netty.NettyRpcServer  isOverTLS 

 

 

PR https://github.com/apache/incubator-tubemq/pull/3

Issue Links

• links to: GitHub Pull Request #608