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

fix(runtime): error messages are now delivered correctly to operate when input validation fails #3704

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

mathias-vandaele
Copy link
Contributor

Description

Add wrapped error message to the overall input validation error

Related issues

#3221

@mathias-vandaele mathias-vandaele requested a review from a team as a code owner November 25, 2024 13:57
@mathias-vandaele mathias-vandaele linked an issue Nov 25, 2024 that may be closed by this pull request
@mathias-vandaele mathias-vandaele self-assigned this Nov 25, 2024
@mathias-vandaele mathias-vandaele added this to the 8.7.0-alpha2 milestone Nov 25, 2024
@mathias-vandaele mathias-vandaele force-pushed the 3221-missing-error-messages-during-request-mapping branch from 8ba3303 to 41851b4 Compare November 26, 2024 08:15
@sbuettner
Copy link
Contributor

As discussed in the architecture session

  • Analyze the output for different constraint violations to see whether we leak any sensitive information
  • Capture the replaced secrets in the outbound context and re-use them to filter out any potential secret values that might be part of a validation message.

@sbuettner sbuettner marked this pull request as draft December 3, 2024 13:46
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.

Missing error messages, during request mapping
2 participants