Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add wrap_exceptions option to exception handler. #363

Merged
merged 2 commits into from
Jan 23, 2025
Merged

Conversation

cofin
Copy link
Member

@cofin cofin commented Jan 23, 2025

When wrap_exceptions is False, the original SQLAlchemy error message will be raised instead of the wrapped Repository error

Fixes #356 (Bug: wrap_sqlalchemy_exception masks db errors)

Copy link

sonarqubecloud bot commented Jan 23, 2025

Copy link

Documentation preview will be available shortly at https://litestar-org.github.io/advanced-alchemy-docs-preview/363

@cofin cofin disabled auto-merge January 23, 2025 16:26
@cofin cofin merged commit ce1f26a into main Jan 23, 2025
20 checks passed
@cofin cofin deleted the cofin/issue356 branch January 23, 2025 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: wrap_sqlalchemy_exception masks db errors
1 participant