sergehuber opened a new pull request, #787:
URL: https://github.com/apache/unomi/pull/787

   ## Summary
   
   - Map `IllegalArgumentException` (rules) and `BadSegmentConditionException` 
(segments) to HTTP **400** `badRequest` via dedicated JAX-RS exception mappers, 
extending the master `AbstractRestExceptionMapper` pattern from #771/#777 
rather than backporting 3-dev monoliths.
   - Update `RuntimeExceptionMapper` so wrapped validation causes also return 
400 (WARN log) instead of 500.
   - Add unit tests in `RestExceptionMapperTest` for direct and wrapped 
validation failures.
   
   ## Context (Phase 2 backport audit)
   
   - **ยง2 audit:** No diff on `RulesServiceEndPoint` / `SegmentServiceEndPoint` 
vs `unomi-3-dev` โ€” service-layer validation already exists on master; this PR 
is **FRESH** REST error-mapping only.
   - **Stack:** Builds on merged #786 (`df5f313de`). No endpoint or validation 
logic changes.
   
   ## Test plan
   
   - [x] `mvn -pl rest test -Dtest=RestExceptionMapperTest` (11 tests pass)
   - [ ] POST invalid rule condition โ†’ 400 `badRequest` (not 500)
   - [ ] POST invalid segment condition โ†’ 400 `badRequest` (not 500)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to