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

Bug #13916-master: fixing embedded images referenced in the email content that were not processed when the email content was extracted. #1316

Closed
wants to merge 1 commit into from

Conversation

SilverYoCha
Copy link
Member

No description provided.

FileUtil.getMimeType(file.getAbsolutePath())).getSubType();
final String fileNameWithExtension = fileName + "." + fileExtension;
final File fileWithExtension = new File(dir, fileNameWithExtension);
Files.move(file.toPath(), fileWithExtension.toPath());

Check failure

Code scanning / SonarCloud

I/O function calls should not be vulnerable to path injection attacks

<!--SONAR_ISSUE_KEY:AYz4XWhOgMjQ58YLgl5O-->Change this code to not construct the path from user-controlled data. <p>See more on <a href="https://sonarcloud.io/project/issues?id=Silverpeas_Silverpeas-Core2&issues=AYz4XWhOgMjQ58YLgl5O&open=AYz4XWhOgMjQ58YLgl5O&pullRequest=1316">SonarCloud</a></p>
…at were not processed when the email content was extracted.
Copy link

sonarcloud bot commented Jan 17, 2024

@mmoqui
Copy link
Member

mmoqui commented Jan 19, 2024

Merged after a rebase by hand with master

@mmoqui mmoqui closed this Jan 19, 2024
@SilverYoCha SilverYoCha deleted the bug-13916-master branch January 25, 2024 09:21
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.

None yet

2 participants