Commit 9649fc49bbdd5eeb0e190c8cdb01d641bd397673
Merge branch '20230427' into 'master_dev'
fix: 鉴权脚本有效性校验完善 See merge request yunteng/thingskit!187
Showing
1 changed file
with
3 additions
and
3 deletions
@@ -34,6 +34,7 @@ import io.netty.channel.ChannelHandlerContext; | @@ -34,6 +34,7 @@ import io.netty.channel.ChannelHandlerContext; | ||
34 | import io.netty.channel.ChannelInboundHandlerAdapter; | 34 | import io.netty.channel.ChannelInboundHandlerAdapter; |
35 | import io.netty.handler.codec.mqtt.*; | 35 | import io.netty.handler.codec.mqtt.*; |
36 | import io.netty.handler.ssl.SslHandler; | 36 | import io.netty.handler.ssl.SslHandler; |
37 | +import io.netty.util.CharsetUtil; | ||
37 | import io.netty.util.ReferenceCountUtil; | 38 | import io.netty.util.ReferenceCountUtil; |
38 | import io.netty.util.concurrent.Future; | 39 | import io.netty.util.concurrent.Future; |
39 | import io.netty.util.concurrent.GenericFutureListener; | 40 | import io.netty.util.concurrent.GenericFutureListener; |
@@ -303,7 +304,7 @@ public class TcpTransportHandler extends ChannelInboundHandlerAdapter implements | @@ -303,7 +304,7 @@ public class TcpTransportHandler extends ChannelInboundHandlerAdapter implements | ||
303 | Futures.addCallback(item, new FutureCallback<String>() { | 304 | Futures.addCallback(item, new FutureCallback<String>() { |
304 | @Override | 305 | @Override |
305 | public void onSuccess(@Nullable String result) { | 306 | public void onSuccess(@Nullable String result) { |
306 | - processAuthTokenConnect(ctx, id, JacksonUtil.fromString(result, TcpAuthEntry.class)); | 307 | + processAuthTokenConnect(ctx, id, JacksonUtil.fromString(result.replace("\\","\\\\"), TcpAuthEntry.class)); |
307 | } | 308 | } |
308 | 309 | ||
309 | @Override | 310 | @Override |
@@ -319,14 +320,13 @@ public class TcpTransportHandler extends ChannelInboundHandlerAdapter implements | @@ -319,14 +320,13 @@ public class TcpTransportHandler extends ChannelInboundHandlerAdapter implements | ||
319 | private void processAuthTokenConnect(ChannelHandlerContext ctx, UUID scriptId, TcpAuthEntry accessToken) { | 320 | private void processAuthTokenConnect(ChannelHandlerContext ctx, UUID scriptId, TcpAuthEntry accessToken) { |
320 | 321 | ||
321 | log.debug("[{}][{}] Processing connect msg for client with user name: {}!", address, sessionId, accessToken); | 322 | log.debug("[{}][{}] Processing connect msg for client with user name: {}!", address, sessionId, accessToken); |
322 | - TransportProtos.ValidateDeviceTokenRequestMsg.Builder request = TransportProtos.ValidateDeviceTokenRequestMsg.newBuilder(); | ||
323 | if (null != accessToken.getClientId()) { | 323 | if (null != accessToken.getClientId()) { |
324 | } | 324 | } |
325 | if (null != accessToken.getUserName()) { | 325 | if (null != accessToken.getUserName()) { |
326 | } | 326 | } |
327 | String token = accessToken.getPassword(); | 327 | String token = accessToken.getPassword(); |
328 | if(StringUtils.isNotEmpty(token)){ | 328 | if(StringUtils.isNotEmpty(token)){ |
329 | - token.trim(); | 329 | + TransportProtos.ValidateDeviceTokenRequestMsg.Builder request = TransportProtos.ValidateDeviceTokenRequestMsg.newBuilder(); |
330 | request.setToken(token); | 330 | request.setToken(token); |
331 | transportService.process(DeviceTransportType.TCP, request.build(), | 331 | transportService.process(DeviceTransportType.TCP, request.build(), |
332 | new TransportServiceCallback<>() { | 332 | new TransportServiceCallback<>() { |