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

Remove non-empty filename check in ContentDisposition builder #30537

Conversation

Foolkin
Copy link

@Foolkin Foolkin commented May 24, 2023

The ContentDisposition Builder must be consistent with ContentDisposition and accept a nullable filename.

The ContentDisposition Builder must be consistent with ContentDisposition and accept nullable filename
@pivotal-cla
Copy link

@Foolkin Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged or decided on label May 24, 2023
@pivotal-cla
Copy link

@Foolkin Thank you for signing the Contributor License Agreement!

@sbrannen sbrannen changed the title ContentDisposition Builder drop nonNull filename check Remove non-null filename check in ContentDisposition builder May 24, 2023
@sbrannen sbrannen added in: web Issues in web modules (web, webmvc, webflux, websocket) type: enhancement A general enhancement labels May 24, 2023
@sbrannen sbrannen changed the title Remove non-null filename check in ContentDisposition builder Remove non-empty filename check in ContentDisposition builder May 24, 2023
@sbrannen sbrannen self-assigned this May 24, 2023
@sbrannen sbrannen removed the status: waiting-for-triage An issue we've not yet triaged or decided on label May 24, 2023
@sbrannen sbrannen added this to the 6.0.10 milestone May 24, 2023
@sbrannen sbrannen closed this in 7f9349b May 24, 2023
@sbrannen
Copy link
Member

sbrannen commented May 24, 2023

Hi @Foolkin,

Congratulations on submitting your first PR to the Spring Framework! 👍

This has been merged into 6.0.x and main.

Thanks

@Foolkin Foolkin deleted the content-disposition-builder-drop-notnull-assertion branch May 24, 2023 12:10
mdeinum pushed a commit to mdeinum/spring-framework that referenced this pull request Jun 29, 2023
This commit ensures that the ContentDisposition.Builder is consistent
with ContentDisposition and accepts null/empty filenames.

Closes spring-projectsgh-30537
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: web Issues in web modules (web, webmvc, webflux, websocket) type: enhancement A general enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants