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.

Sign up for GitHub

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

Jump to bottom

getConfig/getDmmf: Clarify error messages on Rust panics #14006

Closed
jkomyno opened this issue Jun 24, 2022 · 1 comment
Closed

getConfig/getDmmf: Clarify error messages on Rust panics #14006

jkomyno opened this issue Jun 24, 2022 · 1 comment
Assignees
@jkomyno
Labels
team/schema Issue for team Schema.
Milestone
4.0.0

Comments

@jkomyno
Copy link
Contributor

jkomyno commented Jun 24, 2022

The current error message displayed on Rust panics via getConfig and getDmmf may prevent a user to understand the exact source of the error itself. While we should continue augmenting the error to provide additional context to the user, the original error should still be displayed for transparency.

Related issue: #13717.
@jkomyno jkomyno added the team/schema Issue for team Schema. label Jun 24, 2022
@jkomyno jkomyno self-assigned this Jun 24, 2022
@jkomyno jkomyno mentioned this issue Jun 24, 2022
Merged
@jkomyno
Copy link
Contributor Author

jkomyno commented Jun 24, 2022

Closed by #13736.

@jkomyno jkomyno closed this as completed Jun 24, 2022
@janpio janpio added this to the 4.0.0 milestone Jun 25, 2022
@janpio janpio changed the title Clarify error messages on Rust panics getConfig/getDmmf: Clarify error messages on Rust panics Jun 25, 2022
@jkomyno jkomyno mentioned this issue Jun 27, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jkomyno jkomyno

Labels
team/schema Issue for team Schema.
Projects
None yet
Milestone
4.0.0
Development

No branches or pull requests
2 participants
@janpio @jkomyno