mirror of
				https://github.com/continew-org/continew-admin.git
				synced 2025-10-31 00:57:13 +08:00 
			
		
		
		
	chore: continew-starter 2.5.1 => 2.5.2
接口文档显示优化 增强 SQL 注入防御
This commit is contained in:
		| @@ -19,11 +19,11 @@ package top.continew.admin.system.service.impl; | ||||
| import cn.crane4j.annotation.AutoOperate; | ||||
| import cn.hutool.core.bean.BeanUtil; | ||||
| import cn.hutool.core.collection.CollUtil; | ||||
| import cn.hutool.core.lang.Opt; | ||||
| import cn.hutool.core.text.CharSequenceUtil; | ||||
| import cn.hutool.core.util.StrUtil; | ||||
| import com.baomidou.mybatisplus.core.conditions.query.QueryWrapper; | ||||
| import com.baomidou.mybatisplus.core.metadata.IPage; | ||||
| import com.baomidou.mybatisplus.extension.plugins.pagination.Page; | ||||
| import jakarta.servlet.http.HttpServletResponse; | ||||
| import lombok.RequiredArgsConstructor; | ||||
| import lombok.extern.slf4j.Slf4j; | ||||
| @@ -67,7 +67,8 @@ public class LogServiceImpl implements LogService { | ||||
|     @Override | ||||
|     public PageResp<LogResp> page(LogQuery query, PageQuery pageQuery) { | ||||
|         QueryWrapper<LogDO> queryWrapper = this.buildQueryWrapper(query); | ||||
|         IPage<LogResp> page = baseMapper.selectLogPage(pageQuery.toPage(), queryWrapper); | ||||
|         IPage<LogResp> page = baseMapper.selectLogPage(new Page<>(pageQuery.getPage(), pageQuery | ||||
|             .getSize()), queryWrapper); | ||||
|         return PageResp.build(page); | ||||
|     } | ||||
|  | ||||
| @@ -132,12 +133,12 @@ public class LogServiceImpl implements LogService { | ||||
|      * @param sortQuery    排序查询条件 | ||||
|      */ | ||||
|     private void sort(QueryWrapper<LogDO> queryWrapper, SortQuery sortQuery) { | ||||
|         Sort sort = Opt.ofNullable(sortQuery).orElseGet(SortQuery::new).getSort(); | ||||
|         for (Sort.Order order : sort) { | ||||
|             if (null != order) { | ||||
|                 String property = order.getProperty(); | ||||
|                 queryWrapper.orderBy(true, order.isAscending(), CharSequenceUtil.toUnderlineCase(property)); | ||||
|             } | ||||
|         if (sortQuery == null || sortQuery.getSort().isUnsorted()) { | ||||
|             return; | ||||
|         } | ||||
|         for (Sort.Order order : sortQuery.getSort()) { | ||||
|             String property = order.getProperty(); | ||||
|             queryWrapper.orderBy(true, order.isAscending(), CharSequenceUtil.toUnderlineCase(property)); | ||||
|         } | ||||
|     } | ||||
|  | ||||
|   | ||||
| @@ -21,6 +21,7 @@ import cn.hutool.core.bean.BeanUtil; | ||||
| import cn.hutool.core.collection.CollUtil; | ||||
| import com.baomidou.mybatisplus.core.conditions.query.QueryWrapper; | ||||
| import com.baomidou.mybatisplus.core.metadata.IPage; | ||||
| import com.baomidou.mybatisplus.extension.plugins.pagination.Page; | ||||
| import lombok.RequiredArgsConstructor; | ||||
| import org.springframework.stereotype.Service; | ||||
| import org.springframework.transaction.annotation.Transactional; | ||||
| @@ -32,7 +33,7 @@ import top.continew.admin.system.model.resp.MessageResp; | ||||
| import top.continew.admin.system.service.MessageService; | ||||
| import top.continew.admin.system.service.MessageUserService; | ||||
| import top.continew.starter.core.util.validate.CheckUtils; | ||||
| import top.continew.starter.data.mybatis.plus.query.QueryWrapperHelper; | ||||
| import top.continew.starter.data.mybatis.plus.util.QueryWrapperHelper; | ||||
| import top.continew.starter.extension.crud.model.query.PageQuery; | ||||
| import top.continew.starter.extension.crud.model.resp.PageResp; | ||||
|  | ||||
| @@ -54,10 +55,11 @@ public class MessageServiceImpl implements MessageService { | ||||
|     @Override | ||||
|     @AutoOperate(type = MessageResp.class, on = "list") | ||||
|     public PageResp<MessageResp> page(MessageQuery query, PageQuery pageQuery) { | ||||
|         QueryWrapper<MessageDO> queryWrapper = QueryWrapperHelper.build(query); | ||||
|         QueryWrapper<MessageDO> queryWrapper = QueryWrapperHelper.build(query, pageQuery.getSort()); | ||||
|         queryWrapper.apply(null != query.getUserId(), "t2.user_id={0}", query.getUserId()) | ||||
|             .apply(null != query.getIsRead(), "t2.is_read={0}", query.getIsRead()); | ||||
|         IPage<MessageResp> page = baseMapper.selectPageByUserId(pageQuery.toPage(), queryWrapper); | ||||
|         IPage<MessageResp> page = baseMapper.selectPageByUserId(new Page<>(pageQuery.getPage(), pageQuery | ||||
|             .getSize()), queryWrapper); | ||||
|         return PageResp.build(page); | ||||
|     } | ||||
|  | ||||
|   | ||||
| @@ -38,7 +38,7 @@ import top.continew.starter.cache.redisson.util.RedisUtils; | ||||
| import top.continew.starter.core.constant.StringConstants; | ||||
| import top.continew.starter.core.util.validate.CheckUtils; | ||||
| import top.continew.starter.core.util.validate.ValidationUtils; | ||||
| import top.continew.starter.data.mybatis.plus.query.QueryWrapperHelper; | ||||
| import top.continew.starter.data.mybatis.plus.util.QueryWrapperHelper; | ||||
|  | ||||
| import java.util.List; | ||||
| import java.util.Map; | ||||
|   | ||||
| @@ -40,6 +40,7 @@ import com.baomidou.mybatisplus.core.conditions.query.QueryWrapper; | ||||
| import com.baomidou.mybatisplus.core.metadata.IPage; | ||||
| import com.baomidou.mybatisplus.core.toolkit.Wrappers; | ||||
| import com.baomidou.mybatisplus.core.toolkit.support.SFunction; | ||||
| import com.baomidou.mybatisplus.extension.plugins.pagination.Page; | ||||
| import jakarta.annotation.Resource; | ||||
| import jakarta.servlet.http.HttpServletResponse; | ||||
| import lombok.RequiredArgsConstructor; | ||||
| @@ -116,7 +117,9 @@ public class UserServiceImpl extends BaseServiceImpl<UserMapper, UserDO, UserRes | ||||
|     @Override | ||||
|     public PageResp<UserResp> page(UserQuery query, PageQuery pageQuery) { | ||||
|         QueryWrapper<UserDO> queryWrapper = this.buildQueryWrapper(query); | ||||
|         IPage<UserDetailResp> page = baseMapper.selectUserPage(pageQuery.toPage(), queryWrapper); | ||||
|         super.sort(queryWrapper, pageQuery); | ||||
|         IPage<UserDetailResp> page = baseMapper.selectUserPage(new Page<>(pageQuery.getPage(), pageQuery | ||||
|             .getSize()), queryWrapper); | ||||
|         PageResp<UserResp> pageResp = PageResp.build(page, super.getListClass()); | ||||
|         pageResp.getList().forEach(this::fill); | ||||
|         return pageResp; | ||||
|   | ||||
		Reference in New Issue
	
	Block a user