If query arguments are included in a remote stream URI the test for file extensions will fail.

Patch to fix this follows.

CommentFileSizeAuthor
#3 1472378-remote_stream_wrapper-strip_query_args-3.patch657 byteselliotttf
#3 1472378-remote_stream_wrapper-strip_query_args-3.patch657 byteselliotttf
#1 1472378-remote_stream_wrapper-strip_query_args.patch570 byteselliotttf

Comments

elliotttf’s picture

elliotttf commented
StatusFileSize
new 1472378-remote_stream_wrapper-strip_query_args.patch570 bytes

Patch to strip query arguments from file basenames.

elliotttf’s picture

elliotttf commented
Status: Active » Needs review
elliotttf’s picture

elliotttf commented
StatusFileSize
new 1472378-remote_stream_wrapper-strip_query_args-3.patch657 bytes
new 1472378-remote_stream_wrapper-strip_query_args-3.patch657 bytes

Missed a line on the first patch, here's a correct version.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Needs review » Fixed

Thanks, I committed this using $path = parse_url($uri, PHP_URL_PATH) which I prefer to use over regex if possible.
http://drupalcode.org/project/remote_stream_wrapper.git/commit/6cc8a9e

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.