/passport-http-oauth/strategies/utils:originalURL method.

[andriikadyrov90]

First of all thanks for the library. It`s very easy to implement 'oauth' server provider side with it.

Recently I had a task of creating several routes with the same beginning part of the url. For instance, the start of all urls is '/base' and every route under the base must be checked with 'oauth' (ConsumerStrategy from the library) passport authentication middleware, like the following:

app.use('/base', passport.authenticate('oauth', {session: false}), require('./oauth-api'));

in the 'oauth-api' file I create a Route and add 8 urls. So, the complete routes look like this:

router.get('/route1', controller.route1); router.get('/routeN', controller.routeN);
When an oauth Consumer prepares a request to my server, it creates oauth_signature. The URL part of the signature would be like the following:

http://localhost:{port}/base/route1

Whereas the library would consider the url without the base, like this:

http://localhost:{port}/route1

And therefore the signatures don`t match.

The solution that worked for me was to amend ,mentioned in the topic, the 'originalURL' the line

path = req.url || '';

to the line

path = req.originalUrl || '';

[sambrenner]

This appears to be fixed by PR #10, would it be possible to have it merged in?

[LittleCodeGeek]

Seeing as there is no activity on #10, I created #17 with code that more closely resemble the author's.

[ramesius]

Ran into this issue recently when using express Router. @zentarul any chance on finalising your PR #17?

[LittleCodeGeek]

@ramesius I'd love to, but if I'm not mistaken, I need the repository owner to approve the pr first.

[ramesius]

CC @jaredhanson, any chance on getting this mentioned PR (#17) sorted out?

[djromberg]

I just stumbled over the same issue where I want to use express Router but once I do this, the signature verification fails due to the wrong URL. utils.js:29 should use req.originalUrl instead of req.url.