1. Aristarkh Zagorodnikov
  2. mod_gridfs
  3. Pull requests

Pull requests

#1 Merged at 9c26d8f
Repository
Deleted repository
Branch
default (609245e98a49)
Repository
mod_gridfs
Branch
v0.2

Implement context awareness in mod_gridfs

Author
  1. Stafford Brunk
Reviewers
Description

mod_gridfs as it stands now assumes the whole uri is the filename parameter in gridfs. In my case, I am serving images out of a subdirectory within a Rails application and needed GridFS mounted within a Location. This patch allows mod_gridfs to assume the filename parameter is the part of the uri that follows the current location. This is more inline with how nginx-gridfs works. For example:

Location / => foobar.com/foo/bar/baz.jpg filename => foo/bar/baz.jpg

Location /foo/bar => foobar.com/foo/bar/baz.jpg filename => baz.jpg

Comments (1)

  1. Aristarkh Zagorodnikov repo owner

    Thanks for the patch, I'll merge it next week (out of the office at the moment). At a glance it looks fine except the extra std::string copying (I'll take care of it when merging).