When attempting to index files which use the private:// stream wrapper the indexing fails because Drupal returns a 403 to the process requesting the file. The attached patch allows for the file to be read directly from the filesystem instead.

CommentFileSizeAuthor
#2 read_files_from_filesystem-1280730-2.patch2.19 KBtekante
#1 read_files_from_filesystem-1280730-1.patch2.18 KBtekante

Comments

tekante’s picture

tekante commented
StatusFileSize
new read_files_from_filesystem-1280730-1.patch2.18 KB
tekante’s picture

tekante commented
StatusFileSize
new read_files_from_filesystem-1280730-2.patch2.19 KB

The patch in #1 inadvertently switched the extracted text from text mode to xml. This patch corrects that.

jax’s picture

jax commented

This is related to #1148162: Use real path instead of url to retrieve file

ken hawkins’s picture

ken hawkins commented
Status: Active » Needs review

Had same issue on a private file system and tekante's patch in #2 worked for us.

ken hawkins’s picture

ken hawkins commented
Title: Use local filesystem path so that files which are private can be indexed » Allow attachment indexing in private file system

Cleaning up title.

teekay78’s picture

teekay78 commented

#2 works here, too. Thanks!

gaëlg’s picture

gaëlg
he/him
Primary language French
Location Lille, France
commented
Status: Needs review » Reviewed & tested by the community
mike503’s picture

mike503 commented

this is more or less duplicated from #1148162, the only difference is the file:// in the system command (which I don't think is necessary, as the get_realpath() should handle that

izus’s picture

izus commented
Status: Reviewed & tested by the community » Closed (duplicate)

closing as duplicate of #1148162: Use real path instead of url to retrieve file taht got merged.
please feel free to open back if you tested with the last code base and think you find that we are missing something here !
Thanks