Skip to content

Added SAFE_EXCEPTION setting for strict error handling#7478

Open
aKlimau wants to merge 1 commit intopulp:mainfrom
aKlimau:add-exception-handling
Open

Added SAFE_EXCEPTION setting for strict error handling#7478
aKlimau wants to merge 1 commit intopulp:mainfrom
aKlimau:add-exception-handling

Conversation

@aKlimau
Copy link
Contributor

@aKlimau aKlimau commented Mar 17, 2026

Added SAFE_EXCEPTION setting (defaults to False) for strict error handling, returning "Internal error" for every non PulpException if set to True.
Added deprecation warning for every non PulpException error message.

📜 Checklist

  • Commits are cleanly separated with meaningful messages (simple features and bug fixes should be squashed to one commit)
  • A changelog entry or entries has been added for any significant changes
  • Follows the Pulp policy on AI Usage
  • (For new features) - User documentation and test coverage has been added

See: Pull Request Walkthrough

@aKlimau aKlimau marked this pull request as draft March 17, 2026 13:50
@aKlimau aKlimau force-pushed the add-exception-handling branch 2 times, most recently from bb7ebbb to e2033a0 Compare March 18, 2026 09:59
@aKlimau aKlimau marked this pull request as ready for review March 19, 2026 12:17
@aKlimau aKlimau force-pushed the add-exception-handling branch 5 times, most recently from 882f41c to be20dc7 Compare March 20, 2026 13:30
@aKlimau aKlimau force-pushed the add-exception-handling branch from be20dc7 to 791089e Compare March 20, 2026 13:30
@mdellweg mdellweg enabled auto-merge (rebase) March 20, 2026 13:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants