Skip to content

GitLab

  • Menu
Projects Groups Snippets
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • H holdingk
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 0
    • Issues 0
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 4
    • Merge requests 4
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Packages & Registries
    • Packages & Registries
    • Package Registry
    • Infrastructure Registry
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • knitting
  • holdingk
  • Merge requests
  • !1626

Merged
Created Sep 15, 2026 by wangzp@wangzpOwner0 of 2 tasks completed0/2 tasks

H-6:PermissionChecker 默认放行(方案)

  • Overview 0
  • Commits 12
  • Changes 63

H-6 PermissionChecker 默认放行(CWE-862 / CWE-285)。叠在 H-13 上。

Summary

  • 方案:废弃 ModuleCode.ALL 假鉴权;超管走同一套 check;写方法无注解可由 holdingk.permission.write-default-deny 默认拒绝(各环境默认 false)
  • 危险写/导出补真实 sys_button code;升 holdingk-web-config-repo(template + tenant-test)
  • 本 MR 当前为方案文档;拦截器与 P0 接口实现继续提交

Test plan

  • 方案评审:写默认拒绝默认关;超管不跳过 check;GET 危险须显式注解
  • 实现后:无注解 POST+开关 true → 403;超管业务按钮 200;allowSuperAdmin=false 拒绝

无 Maven 升版、无 Flyway。方案:钉钉 H-6

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: hotfix/h6-permission-default-deny