Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Fix redirection to /active #190

Merged
merged 2 commits into from

3 participants

@jackdbernier

No description provided.

@drudge drudge merged commit 6db47cd into Automattic:master
@jackdbernier

Do you plan to publish a new version on npm ?

Thanks

@drudge
Owner

@jackdbernier 0.6.0 was just published to npm.

@rosbo

But now, it doesn't work if you use a path for the kue app. The kue app has no knowledge of where it was mounted. Thus, we can't assume that '/active' is the right path.

For example, if we have
app.use '/kue', kue.app

If we go to /kue, it redirects to /active instead of kue/active

@rosbo

I opened a PR at Automattic/kue#193

@jackdbernier

I fixed it in this PR #195

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Mar 17, 2013
  1. @jackdbernier

    fix redirect to /active

    jackdbernier authored
  2. @jackdbernier

    update version

    jackdbernier authored
This page is out of date. Refresh to see the latest.
Showing with 2 additions and 2 deletions.
  1. +1 −1  lib/http/index.js
  2. +1 −1  package.json
View
2  lib/http/index.js
@@ -65,7 +65,7 @@ app.post('/job', provides('json'), express.bodyParser(), json.createJob);
// routes
app.get('/', function (req, res) {
- res.redirect('active')
+ res.redirect('/active')
});
app.get('/active', routes.jobs('active'));
app.get('/inactive', routes.jobs('inactive'));
View
2  package.json
@@ -1,6 +1,6 @@
{
"name": "kue"
- , "version": "0.5.0"
+ , "version": "0.5.1"
, "description": "Feature rich priority job queue backed by redis"
, "homepage": "http://learnboost.github.com/kue/"
, "keywords": ["job", "queue", "worker", "redis"]
Something went wrong with that request. Please try again.