Skip to content

FormFeature -Added exception for invalid content-disposition header #7525

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

Conversation

JulijaRamoskiene
Copy link
Contributor

Fix for: #7497
Added user-friendly exception when content-disposition header value is not valid.

@dnfclas
Copy link

dnfclas commented Feb 13, 2019

CLA assistant check
All CLA requirements met.

@Tratcher Tratcher self-assigned this Feb 13, 2019
@natemcmaster
Copy link
Contributor

/azp help

@azure-pipelines
Copy link

Supported commands
     help:
          Get descriptions, examples and documentation about supported commands
          Example: help "command_name"
     run:
          Run all pipelines or a specific pipeline for this repository using a comment. Use
          this command by itself to trigger all related pipelines, or specify a pipeline
          to run.
          Example: "run" or "run pipeline_name"

See additional documentation.

@natemcmaster
Copy link
Contributor

/azp run AspNetCore-benchmarking-pr

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@natemcmaster
Copy link
Contributor

/azp run aspnet/AspNetCore/AspNetCore-benchmarking-pr

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@natemcmaster
Copy link
Contributor

Hmm, not sure why it can't find this pipeline: https://dev.azure.com/dnceng/public/_build/index?definitionId=304&_a=completed

Tratcher and others added 3 commits February 14, 2019 10:14
Co-Authored-By: JulijaRamoskiene <Julija.Ramoskiene@bentley.com>
Co-Authored-By: JulijaRamoskiene <Julija.Ramoskiene@bentley.com>
@jkotalik jkotalik removed their request for review February 15, 2019 17:21
@Tratcher Tratcher merged commit e5f4400 into dotnet:master Feb 21, 2019
@Tratcher
Copy link
Member

Thanks

@Tratcher Tratcher added this to the 3.0.0-preview4 milestone Feb 21, 2019
@amcasey amcasey added area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions and removed area-runtime labels Aug 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants