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

#212 implemented MediaFileRepository to replace MediaFileDao #319

Merged
merged 50 commits into from
Dec 3, 2023

Conversation

kagemomiji
Copy link
Owner

@@ -155,7 +155,11 @@
MonitoredMultipartFile monitoredFile = new MonitoredMultipartFile(file, listener);

if (!monitoredFile.isEmpty()) {
String fileName = monitoredFile.getOriginalFilename();
Path filePath = Paths.get(monitoredFile.getOriginalFilename()).normalize();

Check failure

Code scanning / CodeQL

Uncontrolled data used in path expression

This path depends on a [user-provided value](1).
findByFolderInAndMediaTypeAndArtistAndPresentTrue
method signature
getHighestRatedAlbums method
findByFolderInAndMediaTypeAndArtistAndTitleAndPresentTrue
method in MediaFileRepository
@kagemomiji kagemomiji enabled auto-merge December 3, 2023 15:18
@kagemomiji kagemomiji merged commit bdbad53 into main Dec 3, 2023
@kagemomiji kagemomiji deleted the issue212-media-file-repository branch December 3, 2023 15:28
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