Skip to content

fix(security): reapply API authorization hardening - #158

Open
JnyRoad wants to merge 1 commit into
mainfrom
fix/reapply-pr152-api-auth-hardening
Open

fix(security): reapply API authorization hardening#158
JnyRoad wants to merge 1 commit into
mainfrom
fix/reapply-pr152-api-auth-hardening

Conversation

@JnyRoad

@JnyRoad JnyRoad commented Aug 6, 2026

Copy link
Copy Markdown
Owner

Summary

Validation

  • smart: mvn -q -DskipTests clean install -Dmaven.repo.local=... passed after installing the matching current smart-platform-api artifact.
  • smart-module: mvn -q -DskipTests clean compile -Dmaven.repo.local=... passed.
  • smart-h5: pnpm check and pnpm test passed (60 files, 273 tests).
  • smart-ui: pnpm test passed (79 files, 398 tests).
  • smart-app-uniapp: password challenge race and leave current-park contract scripts passed.
  • scripts/security: 2 Node tests passed.

Known validation limits

  • smart-h5 production build requires NEXT_PUBLIC_SECURITY_ENCODE_KEY, which is not available in this isolated checkout.
  • smart-ui lint has one pre-existing error at src/views/admin/social/index.vue:169 plus existing warnings; that file is not changed by this PR.
  • No production deployment or live DoorLock integration test was performed.

Risks

This reintroduces a broad security change across service routes, clients, Nacos templates, and legacy DoorLock compatibility. Recheck runtime configuration and complete integrated testing before merging.

Reapply PR #152, its direct follow-up, and PR #154 after the temporary rollback in PR #157.\n\nThis restores the pre-#157 security tree while preserving unrelated PR #155 release guards and PR #156 documentation cleanup.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant