basic auth not working about log.io HOT 11 CLOSED

Same problem here on fresh install.
Fixed it by editing the lib/servers.js (/usr/local/lib/node_modules/log.io/lib/servers.js) and adding the following line after line 360:

app.use(express.basicAuth(config.auth.user,config.auth.pass));

It now reads with the line before and after:

staticPath = (_ref = config.staticPath) != null ? _ref : __dirname + '/../';
app.use(express.basicAuth(config.auth.user,config.auth.pass));
return app.use(express"static")

You probably want to add an if/else conditioning to whether the auth already exists.

from log.io.

Same here.

from log.io.

@d4fseeker, should the change in /usr/local/lib/node_modules/log.io/lib/server.js be wrapped in a:

if (config.auth) {
app.use(express.basicAuth(config.auth.user,config.auth.pass));
}

from log.io.

I cannot access a log.io install atm, so you'll have to check but if I remember correctly the wrapper class always exists and thus would return bool/true no matter what.
You'd have to check config.auth.user and config.auth.pass to bypass that, e.g:

if(config.auth.user != null && config.auth.pass != null)
app.use(express.basicAuth(config.auth.user,config.auth.pass));

I have to admit I'm not sure if comparing against null is sufficient since CoffeeScript has made me lazy with it's "?" operator.

from log.io.

To be perfectly equivalent with Coffeescript's "?" Operator, you'd need to check for both null and undefined.

from log.io.

This is the correct syntax:

if (config.auth && config.auth.user && config.auth.pass) {
    app.use(express.basicAuth(config.auth.user,config.auth.pass));
}

from log.io.

and here is the coffee script :

staticPath = config.staticPath ? __dirname + '/../'
app.use express.basicAuth(config.auth.user, config.auth.pass)  if config.auth and config.auth.user and config.auth.pass
app.use express.static staticPath

from log.io.

HI,

For me I fixed it like this (near line 360) file /usr/local/lib/node_modules/log.io/lib/server.js :
staticPath = (_ref = config.staticPath) != null ? _ref : __dirname + '/../';
// add fix start
app.use(express.basicAuth(config.auth.user, config.auth.pass))
if (config.auth && config.auth.user && config.auth.pass)
// fix end
return app.use(express"static");

And It works.

Regards,

from log.io.

Hi, I've downloaded & installed log.io today and still have no auth, my config file looks like:

exports.config = {
host: '0.0.0.0',
port: 28778,

// Enable HTTP Basic Authentication
auth: {
user: "test",
pass: "test123"
},

/*
// Enable HTTPS/SSL
ssl: {
key: '/path/to/privatekey.pem',
cert: '/path/to/certificate.pem'
},
*/

/*
// Restrict access to websocket (socket.io)
// Uses socket.io 'origins' syntax
restrictSocket: ':',
*/

/*
// Restrict access to http server (express)
restrictHTTP: [
"192.168.29.39",
"10.0.*"
]
*/
}

Any hint on why this is happening?

Thanks for your help.

from log.io.

@albertojm, i install by npm today and i have same problom.

if you check /usr/lib/node_modules/log.io/src/server.coffee, you'll find something missing around line 180 and 181:
if @auth?
app.use express.basicAuth @auth.user, @auth.pass

it looks like code in npm is not the same in current repo...?

try add these to /usr/lib/node_modules/log.io/lib/server.js

app = express();
...
   //just add these three line below
   if (this.auth != null) {
          app.use(express.basicAuth(this.auth.user, this.auth.pass));
   }
...
if (config.restrictHTTP) {

then edit config file and run

it works for me!

from log.io.

Thanks @vansteki ! that worked for me too.

from log.io.