Skip to content

Release/v1.0.0 hardening#37

Merged
MarkADom merged 3 commits intodevelopfrom
release/v1.0.0-hardening
Feb 27, 2026
Merged

Release/v1.0.0 hardening#37
MarkADom merged 3 commits intodevelopfrom
release/v1.0.0-hardening

Conversation

@MarkADom
Copy link
Copy Markdown
Owner

Summary

Standardized action/message responses to ApiMessageResponseDTO across key controllers, including root/home and admin cache clear endpoints, improving response consistency for clients.
Hardened actuator/security behavior by limiting dev actuator exposure and enforcing EndpointRequest rules (health/info public, all other actuator endpoints ADMIN-only).
Strengthened notification permission checks in service layer with actor-context validation across board/project/task contexts and admin override; also tightened markAllAsRead actor/target scope rules.
Improved deprecation hygiene by marking email-path aliases as deprecated (Use /me variant), keeping principal-equality guards, delegating to /me handlers, and hiding deprecated aliases from OpenAPI.

Type of change

  • feat
  • fix
  • refactor
  • docs
  • test
  • ci

Validation

  • I ran local checks
  • I included commands/results below

Commands run:

 ./gradlew test detekt jacocoTestReport sonarqube 

Docs impact

  • No docs change needed
  • Updated README/docs
  • Updated API tests (Bruno)

Security impact

  • No security impact
  • Security-relevant change explained below

Actuator exposure in application-dev.yml is restricted to health,info,env,mappings,metrics.
SecurityConfig uses actuator-aware rules:
- EndpointRequest.to("health","info") => permitAll
- EndpointRequest.toAnyEndpoint() => requires ADMIN
Notification service permission model now validates actor context for board/project/task and enforces non-admin self-scope on markAllAsRead.
Deprecated email-path aliases remain for compatibility but are hidden from OpenAPI and guarded by principal equality checks.

@MarkADom MarkADom self-assigned this Feb 27, 2026
@sonarqubecloud
Copy link
Copy Markdown

@MarkADom MarkADom merged commit 13b1b34 into develop Feb 27, 2026
3 checks passed
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