DMC/studentportalen
1
1
Fork 0
Code Issues 50 Pull Requests 1 Actions Projects 1 Wiki Activity

Exception handling #82

Open
stne3960 wants to merge 6 commits from feature/exception-handling into main
pull from: feature/exception-handling
merge into: DMC:main
Conversation 1 Commits 6 Files Changed 6 +249
stne3960 commented 2026-01-30 11:49:19 +01:00
Owner
Copy Link

This PR adds centralized exception handling with RFC 7807 Problem Details, aligning with https://gitea.dsv.su.se/DMC/api-spec/src/branch/main/daisy/grupprumsbokning.md

It introduces StudentportalenException as the abstract base exception class for all BFF errors. Each subclass defines:

  • getType() - URI identifying the error type (e.g., studentportalen:forbidden)
  • getTitle() - Short human-readable summary
  • getStatus() - HTTP status code

Adds specific exception types: ValidationException, ForbiddenException, and ServiceUnavailableException for now.
Implements GlobalExceptionHandler to convert exceptions to RFC 7807 format.
Adds spring-boot-starter-validation dependency for @Valid support

This PR adds centralized exception handling with RFC 7807 Problem Details, aligning with https://gitea.dsv.su.se/DMC/api-spec/src/branch/main/daisy/grupprumsbokning.md It introduces `StudentportalenException` as the abstract base exception class for all BFF errors. Each subclass defines: - getType() - URI identifying the error type (e.g., `studentportalen:forbidden`) - getTitle() - Short human-readable summary - getStatus() - HTTP status code Adds specific exception types: `ValidationException`, `ForbiddenException`, and `ServiceUnavailableException` for now. Implements `GlobalExceptionHandler` to convert exceptions to RFC 7807 format. Adds `spring-boot-starter-validation` dependency for `@Valid` support
stne3960 added 6 commits 2026-01-30 11:49:20 +01:00
Prettier and checkstyle
All checks were successful
Code style BFF / check (pull_request) Successful in 1m28s
Details
Deploy to branch.dsv.su.se / deploy (pull_request) Successful in 5m19s
Details
9e63eb6406
stne3960 requested review from ansv7779 2026-01-30 11:51:09 +01:00
stne3960 added the
Code Review
 label 2026-01-30 11:51:17 +01:00
gitea-actions commented 2026-01-30 11:54:39 +01:00
First-time contributor
Copy Link

Deployed to https://studentportalen-feature-exception-handling.branch.dsv.su.se

Deployed to https://studentportalen-feature-exception-handling.branch.dsv.su.se
All checks were successful
Code style BFF / check (pull_request) Successful in 1m28s
Details
Deploy to branch.dsv.su.se / deploy (pull_request) Successful in 5m19s
Details
This pull request doesn't have enough required approvals yet. 0 of 1 official approvals granted.
You are not authorized to merge this pull request.
View command line instructions.

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin feature/exception-handling:feature/exception-handling
git checkout feature/exception-handling
Sign in to join this conversation.
Reviewers
No Reviewers
ansv7779
Labels
Clear labels
Code Review
Design
Component
Design
Page
Design Review
Design in progress
Design system
Front end
bug
Something is not working
duplicate
This issue or pull request already exists
enhancement
New feature
help wanted
Need some help
invalid
Something is wrong
priority
question
More information is needed
wontfix
This won't be fixed
No Label
Code Review
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: DMC/studentportalen#82
No description provided.
Delete Branch "feature/exception-handling"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?