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

Patch to fix this follows.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

elliotttf’s picture

Patch to strip query arguments from file basenames.

elliotttf’s picture

Status: Active » Needs review
elliotttf’s picture

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

Dave Reid’s picture

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.