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

EPMRPP-84002 attachment file name #1758

Closed
wants to merge 4 commits into from

Conversation

pbortnik
Copy link
Collaborator

@pbortnik pbortnik commented Aug 7, 2023

No description provided.

@@ -132,8 +132,7 @@ private void toResponse(HttpServletResponse response, BinaryData binaryData) {
response.setContentType(binaryData.getContentType());
if (binaryData.getFileName() != null) {
response.setHeader(HttpHeaders.CONTENT_DISPOSITION,
String.format("attachment; filename=%s", binaryData.getFileName())
);
"attachment; filename=\"" + binaryData.getFileName() + "\"");
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.sizes.LineLengthCheck> reported by reviewdog 🐶
Line is longer than 100 characters (found 117).

@@ -132,8 +132,7 @@ private void toResponse(HttpServletResponse response, BinaryData binaryData) {
response.setContentType(binaryData.getContentType());
if (binaryData.getFileName() != null) {
response.setHeader(HttpHeaders.CONTENT_DISPOSITION,
String.format("attachment; filename=%s", binaryData.getFileName())
);
"attachment; filename=\"" + binaryData.getFileName() + "\"");
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> reported by reviewdog 🐶
Line contains a tab character.

@pbortnik pbortnik closed this Aug 7, 2023
@pbortnik pbortnik deleted the EPMRPP-84002_attachment_file_name branch August 7, 2023 08:14
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.

1 participant