search

go-admin-team / go-admin

基于Gin + Vue + Element UI & Arco Design & Ant Design 的前后端分离权限管理系统脚手架（包含了：多租户的支持，基础用户管理功能，jwt鉴权，代码生成器，RBAC资源控制，表单构建，定时任务等）3分钟构建自己的中后台项目；项目文档》：https://www.go-admin.pro V2 Demo： https://vue2.go-admin.dev V3 Demo： https://vue3.go-admin.dev Antd PRO：https://antd.go-admin.pro
https://www.go-admin.pro
MIT License
11.39k stars 2.37k forks source link

这框架前端写的可以 后端真是火葬场 方法该来该去乱的一笔 构建三分钟 改一个功能要三天 恶心的一笔 #785

Closed QTTQ closed 5 months ago

QTTQ commented 8 months ago

Describe the bug A clear and concise description of what the bug is.

To Reproduce Steps to reproduce the behavior:

  1. Go to '...'
  2. Click on '....'
  3. Scroll down to '....'
  4. See error

Expected behavior A clear and concise description of what you expected to happen.

Screenshots If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

Smartphone (please complete the following information):

Additional context Add any other context about the problem here.

2015WUJI01 commented 8 months ago

前端 bug 也不少，还有 icon 错乱的 bug

whileW commented 8 months ago

api部分 如果 func (e SysApi) GetPage(c gin.Context) 不小心加了一个 变成 func (e SysApi) GetPage(c gin.Context) 内部 MakeContext(c) 可能会导致并发问题，b请求的context会覆盖掉a请求的context

2015WUJI01 commented 8 months ago

api部分

如果

func (e SysApi) GetPage(c gin.Context) 不小心加了一个 变成 func (e SysApi) GetPage(c gin.Context)

内部 MakeContext(c) 可能会导致并发问题，b请求的context会覆盖掉a请求的context

你这么一说还真是，平常没考虑这个问题

lwnmengjing commented 5 months ago

api部分 如果 func (e SysApi) GetPage(c gin.Context) 不小心加了一个 变成 func (e SysApi) GetPage(c gin.Context) 内部 MakeContext(c) 可能会导致并发问题，b请求的context会覆盖掉a请求的context

是的，这个问题确实是存在的，不过mss-boot中对这些问题都做了修复

lwnmengjing commented 5 months ago

api部分 如果 func (e SysApi) GetPage(c gin.Context) 不小心加了一个 变成 func (e SysApi) GetPage(c gin.Context) 内部 MakeContext(c) 可能会导致并发问题，b请求的context会覆盖掉a请求的context

你这么一说还真是，平常没考虑这个问题

所以并发高的话，要慎重用

wenjianzhang commented 5 months ago

Describe the bug A clear and concise description of what the bug is.

To Reproduce Steps to reproduce the behavior:

  1. Go to '...'
  2. Click on '....'
  3. Scroll down to '....'
  4. See error

Expected behavior A clear and concise description of what you expected to happen.

Screenshots If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

  • OS: [e.g. iOS]
  • Browser [e.g. chrome, safari]
  • Version [e.g. 22]

Smartphone (please complete the following information):

  • Device: [e.g. iPhone6]
  • OS: [e.g. iOS8.1]
  • Browser [e.g. stock browser, safari]
  • Version [e.g. 22]

Additional context Add any other context about the problem here.

抱歉go-admin对您造成了困扰，如果有具体的建议也可以提交过来的哦，一起为开源努力！