Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dms/sqle支持统一的swagger server服务 #102

Closed
3 tasks done
rocky114 opened this issue Dec 5, 2023 · 0 comments
Closed
3 tasks done

dms/sqle支持统一的swagger server服务 #102

rocky114 opened this issue Dec 5, 2023 · 0 comments
Assignees
Labels
Milestone

Comments

@rocky114
Copy link
Contributor

rocky114 commented Dec 5, 2023

现状

当前API接口仅在代码中提供了swag.json文件,没有在线swag server 服务

预期

部署服务后,同时提供在线swag server

实现方案

方案一

dms/sqle生成一个swagger文档

优点: 在线swagger服务配置简单,组件文档以一个文档形式展示
缺点: 需要使用脚本合并dms/sqle组件的文档,swagger无合并swagger文档特性且合并文档耦合高,难以维护

方案二

dms/sqle各自生成swagger文档

优点: 文档以下拉框形式选择查看不同组件的文档,dms/sqle组件生成文档的脚本保持不变
缺点: 需要修改swagger服务初始化参数,打包流程繁琐,开发debug调试阶段无法更新swagger文档,只有再次重新打包才能更新swagger文档

方案三

sqle新增获取swagger.yaml文件接口,dms调用该接口获取sqle的swagger.yaml文件

最终效果:

右上角选择dms/sqle:
20240606111750_rec_

通过swgger server调用dms登录接口
20240606112210_rec_

通过swgger server调用sqle白名单GET接口
20240606111925_rec_

使用方式

填入token后即可正常调用接口(dms/sqle使用方式相同)

image

总结

方案三最简单明了,实现成本低,方案一和方案二实现复杂且都存在各自的缺点,综上,最终选择方案三.

todo-list

  • sqle新增获取swagger.yaml文件GET接口 4h
  • dms新增swagger接口 8h
  • 解决点击浏览器刷新404的问题 4h

影响面:

sqle新增 @basepath /sqle 基础api前缀,调用sqle接口时,需要新增 /sqle 前缀,只对swagger调用有影响,对系统集成无影响

// @title Sqle API Docs
// @version 1.0
// @description This is a sample server for dev.
// @securityDefinitions.apikey ApiKeyAuth
// @in header
// @name Authorization
// @BasePath /sqle
@rocky114 rocky114 added this to the v3.2312.0 milestone Dec 5, 2023
@rocky114 rocky114 self-assigned this Dec 5, 2023
@LordofAvernus LordofAvernus removed this from the v3.2312.0 milestone Dec 28, 2023
@ColdWaterLW ColdWaterLW added the sqle 和sqle相关 label Feb 27, 2024
@taolx0 taolx0 assigned taolx0 and unassigned rocky114 Jun 6, 2024
@taolx0 taolx0 changed the title add swagger documentation service dms/sqle支持swagger server服务 Jun 6, 2024
@taolx0 taolx0 changed the title dms/sqle支持swagger server服务 dms/sqle支持统一的swagger server服务 Jun 12, 2024
@taolx0 taolx0 closed this as completed Jun 13, 2024
@LordofAvernus LordofAvernus added this to the v3.2406.0 milestone Jun 14, 2024
LordofAvernus pushed a commit that referenced this issue Jul 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants