omniauth / omniauth / 676

100%

Pull #907

travis-ci

Add support for using developer strategy behind mapping

If the Developer strategy is used behind middleware like this:

map '/api'
  use OmniAuth::Strategies::Developer

  # callback here for /auth/developer/callback
end

the callback path that is used in the form does not contain the
ENV['SCRIPT_NAME'] before the callback_path

Currently the form path would be:
/auth/developer/callback

But since our callback handler is also behind this mapping it
should be:
/api/auth/developer/callback

This fix prepends `script_name` to the `callback_path` to give us
the problem form submission url regardless of whether it's behind
a mapping or not

3 of 3 new or added lines in 1 file covered. (100.0%)

501 of 539 relevant lines covered (92.95%)

101.28 hits per line