修正xxl-job.xml端口自动获取的描述和执行器在指定ip自动获取端口时的错误

1.修正xml配置文件描述信息 int类型不允许xml为空的配置会报NPE
2.修复指定执行器ip时自动获取端口号的问题,指定ip与InetAddress.anyLocalAddress()结果不一致会导致验证端口是否使用错误。
v1.9.2
raoq@vip.qq.com 7 years ago
parent 960b8d5f5a
commit 76b629ac5f

@ -129,7 +129,7 @@ public class XxlJobExecutor implements ApplicationContextAware {
private NetComServerFactory serverFactory = new NetComServerFactory(); private NetComServerFactory serverFactory = new NetComServerFactory();
private void initExecutorServer(int port, String ip, String appName, String accessToken) throws Exception { private void initExecutorServer(int port, String ip, String appName, String accessToken) throws Exception {
// valid param // valid param
port = port>0?port: NetUtil.findAvailablePort(9999); port = port>0?port: NetUtil.findAvailablePort(9999,ip);
// start server // start server
NetComServerFactory.putService(ExecutorBiz.class, new ExecutorBizImpl()); // rpc-service, base on jetty NetComServerFactory.putService(ExecutorBiz.class, new ExecutorBizImpl()); // rpc-service, base on jetty

@ -2,8 +2,10 @@ package com.xxl.job.core.util;
import org.slf4j.Logger; import org.slf4j.Logger;
import org.slf4j.LoggerFactory; import org.slf4j.LoggerFactory;
import org.springframework.util.StringUtils;
import java.io.IOException; import java.io.IOException;
import java.net.InetSocketAddress;
import java.net.ServerSocket; import java.net.ServerSocket;
/** /**
@ -15,23 +17,24 @@ public class NetUtil {
private static Logger logger = LoggerFactory.getLogger(NetUtil.class); private static Logger logger = LoggerFactory.getLogger(NetUtil.class);
/** /**
* find avaliable port * find avaliable port by ip
* *
* @param defaultPort * @param defaultPort
* @param ip
* @return * @return
*/ */
public static int findAvailablePort(int defaultPort) { public static int findAvailablePort(int defaultPort,String ip) {
int portTmp = defaultPort; int portTmp = defaultPort;
while (portTmp < 65535) { while (portTmp < 65535) {
if (!isPortUsed(portTmp)) { if (!isPortUsed(portTmp,ip)) {
return portTmp; return portTmp;
} else { } else {
portTmp++; portTmp++;
} }
} }
portTmp = defaultPort--; portTmp = --defaultPort;
while (portTmp > 0) { while (portTmp > 0) {
if (!isPortUsed(portTmp)) { if (!isPortUsed(portTmp,ip)) {
return portTmp; return portTmp;
} else { } else {
portTmp--; portTmp--;
@ -40,18 +43,34 @@ public class NetUtil {
throw new IllegalStateException("no available port."); throw new IllegalStateException("no available port.");
} }
/**
* find avaliable port
*
* @param defaultPort
* @return
*/
public static int findAvailablePort(int defaultPort) {
return findAvailablePort(defaultPort,null);
}
/** /**
* check port used * check port used
* *
* @param port * @param port
* @param ip InetAddress.anyLocalAddress()
* @return * @return
*/ */
public static boolean isPortUsed(int port) { public static boolean isPortUsed(int port,String ip) {
boolean used = false; boolean used = false;
ServerSocket serverSocket = null; ServerSocket serverSocket = null;
try { try {
if(StringUtils.isEmpty(ip)){
serverSocket = new ServerSocket(port); serverSocket = new ServerSocket(port);
used = false; }else {
serverSocket = new ServerSocket();
serverSocket.bind(new InetSocketAddress(ip,port));
}
} catch (IOException e) { } catch (IOException e) {
logger.debug(">>>>>>>>>>> xxl-job, port[{}] is in use.", port); logger.debug(">>>>>>>>>>> xxl-job, port[{}] is in use.", port);
used = true; used = true;

@ -29,7 +29,7 @@
<property name="appName" value="${xxl.job.executor.appname}" /> <property name="appName" value="${xxl.job.executor.appname}" />
<!-- 执行器IP[选填],为空则自动获取 --> <!-- 执行器IP[选填],为空则自动获取 -->
<property name="ip" value="${xxl.job.executor.ip}" /> <property name="ip" value="${xxl.job.executor.ip}" />
<!-- 执行器端口号[选填],为则自动获取 --> <!-- 执行器端口号[选填],为<0则自动获取 -->
<property name="port" value="${xxl.job.executor.port}" /> <property name="port" value="${xxl.job.executor.port}" />
<!-- 访问令牌[选填],非空则进行匹配校验 --> <!-- 访问令牌[选填],非空则进行匹配校验 -->
<property name="accessToken" value="${xxl.job.accessToken}" /> <property name="accessToken" value="${xxl.job.accessToken}" />

Loading…
Cancel
Save