Skip to content

chore: Align Aviator exception handling with Aviator simple/technical exceptions#1020

Draft
kireetivar wants to merge 1 commit into
fortify:feat/v3.x/aviator/26.3from
kireetivar:p/kireetivar/ex_handle
Draft

chore: Align Aviator exception handling with Aviator simple/technical exceptions#1020
kireetivar wants to merge 1 commit into
fortify:feat/v3.x/aviator/26.3from
kireetivar:p/kireetivar/ex_handle

Conversation

@kireetivar
Copy link
Copy Markdown
Contributor

Summary

  • replace raw RuntimeException and IllegalStateException wrappers at Aviator technical-failure boundaries with AviatorTechnicalException
  • use AviatorSimpleException for direct user/config validation failures in Priority and NumberRangeComparer
  • add focused regression coverage for the changed technical-exception boundaries
  • update the existing NumberRangeComparer test to expect AviatorSimpleException

Details

This change keeps Aviator exception handling aligned with fcli conventions:

  • technical failures remain technical:
    • FPR metadata parsing
    • filtertemplate.xml parse/save failures
    • hashing helper failures
    • gRPC message-to-JSON conversion failures
    • direct invalid input/config validation is surfaced as simple:
    • invalid priority values
    • malformed numeric range syntax

@kireetivar kireetivar self-assigned this May 30, 2026
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