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.

Sign up for GitHub

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

Jump to bottom

创建或修改变量时不允许传非当前实例相关的变量 #68

Closed
jxinging opened this issue Oct 29, 2021 · 0 comments
Closed

创建或修改变量时不允许传非当前实例相关的变量 #68

jxinging opened this issue Oct 29, 2021 · 0 comments
Assignees
@jxinging
Labels
improvement

Comments

@jxinging
Copy link
Contributor

目前版本前端在创建环境或修改环境时会将继承的所有变量以及当前实例的变量传递到后端，后端接收后检查如果变量有 id 则做变量更新，否则创建变量。这里的问题:

  1. 接口入参中可以传 scope 不是当前级别的数据，也会创建对应的变量，变量更新也有这个问题
  2. 后端接口实现时是会自动计算继承的变量的，不需要调用方将所有变量传入
  3. 即使变量内容没有更新也会执行 update

建议修改:

  • 创建或更新实例变量时只传当前实例相关的变量，继承变量不传入（包括环境、云模板、项目、组织的变量设置）
  • 后端接口处理时只允许修改与当前实例相关的变量
  • 变量只在有变化时做更新，而不是每次提交都更新
    • 这个也可以改交互方式，添加和修改变量时每一行单独保存而不是整体提交保存
@jxinging jxinging changed the title 创建或修改实例变量时只需要传当前实例这一级添加的变量 创建或修改变量时不允许传非当前实例相关的变量 Oct 29, 2021
@jxinging jxinging self-assigned this Nov 8, 2021
@jxinging jxinging mentioned this issue Nov 17, 2021
Merged
jxinging added a commit that referenced this issue Nov 18, 2021
@jxinging
Feat: 变量更新接口改为只支持传当前实例添加的变量 (#112)
70d5a3a 
* refactor: 调整创建和更新变量的方式

- 新增接口: /variables/scope/{scope}/{objectId}，用于更新实例的变量列表
- 修改环境创建接口：variables 参数只支持传入环境级别的变量
- 修改环境部署接口：移除 delVariables 参数，variables 参数设计为全量更新, variables 中不存在的变量会被删除

resolve #68
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jxinging jxinging

Labels
improvement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jxinging