Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

URL parsing maybe broken #43

Open
pulkitsinghal opened this issue Jun 5, 2017 · 0 comments
Open

URL parsing maybe broken #43

pulkitsinghal opened this issue Jun 5, 2017 · 0 comments

Comments

@pulkitsinghal
Copy link

pulkitsinghal commented Jun 5, 2017

I do not think that Matador can handle URL parsing when the input is as follows:

var app = require('bull-ui/app')({
  redis: {
    url: 'redis://:password@host:6379'
  }
});

It automatically falls back on 127.0.0.1:6379 after failing to parse the URL. I realized this after I tried to create a job from the /newjob page.

Here are my dependencies:

"dependencies": {
    "bull-ui": "1.2.3"
},

There is obviously a workaround:

var app = require('bull-ui/app')({
  redis: {
    host: 'pulkit-dev-3.shoppinpal.com',
    port: '6379',
    password: 'r3dis'
  }
});

But it thought it worth mentioning the breaking behaviour above.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant