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

Ensure that invalid variable argument doesn't silently crash #9760

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

scbedd
Copy link
Member

@scbedd scbedd commented Feb 1, 2025

Remove FromBody[] everywhere in the app. It silently crashes when an invalid request is sent.

Resolves #9757

todo:

  • Resolve failing .NET integration test runs

@scbedd scbedd self-assigned this Feb 1, 2025
@scbedd scbedd changed the title Ensure that variable argument doesn't silently crash Ensure that invalid variable argument doesn't silently crash Feb 1, 2025
@scbedd scbedd marked this pull request as ready for review February 11, 2025 21:00
@scbedd
Copy link
Member Author

scbedd commented Feb 11, 2025

/azp run tools - test-proxy - tests

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 🔬 Dev in PR
Development

Successfully merging this pull request may close these issues.

Test-Proxy silently crashes on invalid variables input
1 participant