Skip to content

[codex] Improve Spring Boot MySQL application structure#1

Merged
DoRmAmMu1997 merged 1 commit into
mainfrom
codex/springboot-mysql-cleanup
May 10, 2026
Merged

[codex] Improve Spring Boot MySQL application structure#1
DoRmAmMu1997 merged 1 commit into
mainfrom
codex/springboot-mysql-cleanup

Conversation

@DoRmAmMu1997
Copy link
Copy Markdown
Owner

@DoRmAmMu1997 DoRmAmMu1997 commented May 10, 2026

Summary

  • Refactors the REST resources to constructor-injected service classes while preserving the existing /rest/... route shape.
  • Adds shared API error handling, safer missing-record and duplicate-record behavior, password write-only JSON handling, and environment-driven MySQL config.
  • Adds H2-backed Spring tests, Java 11 GitHub Actions CI, .gitignore, Maven wrapper metadata, and a project-specific README explaining structure, setup, routes, and tests.

Test Plan

  • pom.xml parsed successfully as XML locally.
  • git diff --check completed with no whitespace errors.
  • Static scan found no remaining production @Autowired, .findById(...).get(), broad catch(Exception), or removed custom validator references.
  • ./mvnw.cmd test could not run locally because this environment has no Java/JAVA_HOME configured.
  • GitHub Actions Java CI passed on Java 11 for commit ee9290e139bb3d3dab6de740cb4ecb73c6f2b6fa: https://github.com/DoRmAmMu1997/My-SpringBoot-MySQL-Application/actions/runs/25627390275

Notes

  • Existing public endpoint paths are intentionally preserved.
  • Password hashing is out of scope for this educational cleanup pass; password fields are now hidden from JSON responses.

@DoRmAmMu1997 DoRmAmMu1997 force-pushed the codex/springboot-mysql-cleanup branch 2 times, most recently from c6a4fa5 to 5cfd03e Compare May 10, 2026 11:18
Co-authored-by: Codex <codex@openai.com>
@DoRmAmMu1997 DoRmAmMu1997 force-pushed the codex/springboot-mysql-cleanup branch from 5cfd03e to ee9290e Compare May 10, 2026 11:21
@DoRmAmMu1997 DoRmAmMu1997 marked this pull request as ready for review May 10, 2026 11:22
@DoRmAmMu1997 DoRmAmMu1997 merged commit 3d77145 into main May 10, 2026
1 check 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