DATAMONGO-2021 - Use getObjectId() instead of getFilename() for opening the GridFS download stream.

nhh · mp911de · commit e18f506eddb0 · 2018-07-06T13:12:46.000+02:00
Using the file name leads to duplicate resource streams as file names are not unique therefore we're using the file's ObjectId to lookup the file content. Original pull request: spring-projects#581.
diff --git a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/gridfs/GridFsTemplate.java b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/gridfs/GridFsTemplate.java
@@ -240,7 +240,7 @@ public GridFsResource getResource(GridFSFile file) {
 
 		Assert.notNull(file, "GridFSFile must not be null!");
 
-		return new GridFsResource(file, getGridFs().openDownloadStream(file.getFilename()));
+		return new GridFsResource(file, getGridFs().openDownloadStream(file.getObjectId()));
 	}
 
 	/*

Original file line number	Diff line number	Diff line change
`@@ -240,7 +240,7 @@ public GridFsResource getResource(GridFSFile file) {`
`240`	`240`
`241`	`241`	`Assert.notNull(file, "GridFSFile must not be null!");`
`242`	`242`
`243`		`- return new GridFsResource(file, getGridFs().openDownloadStream(file.getFilename()));`
	`243`	`+ return new GridFsResource(file, getGridFs().openDownloadStream(file.getObjectId()));`
`244`	`244`	`}`
`245`	`245`
`246`	`246`	`/*`