2016-09-27 10:55:51 +00:00
|
|
|
expect = require("chai").expect
|
2017-01-16 11:46:59 +00:00
|
|
|
assert = require("chai").assert
|
2016-09-27 10:55:51 +00:00
|
|
|
async = require("async")
|
|
|
|
User = require "./helpers/User"
|
|
|
|
request = require "./helpers/request"
|
|
|
|
settings = require "settings-sharelatex"
|
|
|
|
redis = require "./helpers/redis"
|
2017-01-16 11:46:59 +00:00
|
|
|
_ = require 'lodash'
|
2016-09-27 10:55:51 +00:00
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
# Expectations
|
|
|
|
expectProjectAccess = (user, projectId, callback=(err,result)->) ->
|
|
|
|
# should have access to project
|
|
|
|
user.openProject projectId, (err) =>
|
|
|
|
expect(err).to.be.oneOf [null, undefined]
|
|
|
|
callback()
|
|
|
|
|
|
|
|
expectNoProjectAccess = (user, projectId, callback=(err,result)->) ->
|
|
|
|
# should not have access to project page
|
|
|
|
user.openProject projectId, (err) =>
|
|
|
|
expect(err).to.be.instanceof Error
|
|
|
|
callback()
|
|
|
|
|
|
|
|
# Actions
|
|
|
|
tryLoginThroughRegistrationForm = (user, email, password, callback=(err, response, body)->) ->
|
|
|
|
user.getCsrfToken (err) ->
|
|
|
|
return callback(err) if err?
|
|
|
|
user.request.post {
|
|
|
|
url: "/register"
|
|
|
|
json:
|
|
|
|
email: email
|
|
|
|
password: password
|
|
|
|
}, callback
|
|
|
|
|
|
|
|
|
2017-01-16 11:46:59 +00:00
|
|
|
describe "LoginRateLimit", ->
|
|
|
|
|
|
|
|
before ->
|
|
|
|
@user = new User()
|
|
|
|
@badEmail = 'bademail@example.com'
|
|
|
|
@badPassword = 'badpassword'
|
|
|
|
|
|
|
|
it 'should rate limit login attempts after 10 within two minutes', (done) ->
|
|
|
|
@user.request.get '/login', (err, res, body) =>
|
|
|
|
async.timesSeries(
|
|
|
|
15
|
|
|
|
, (n, cb) =>
|
|
|
|
@user.getCsrfToken (error) =>
|
|
|
|
return cb(error) if error?
|
|
|
|
@user.request.post {
|
|
|
|
url: "/login"
|
|
|
|
json:
|
|
|
|
email: @badEmail
|
|
|
|
password: @badPassword
|
|
|
|
}, (err, response, body) =>
|
|
|
|
cb(null, body?.message?.text)
|
|
|
|
, (err, results) =>
|
|
|
|
# ten incorrect-credentials messages, then five rate-limit messages
|
|
|
|
expect(results.length).to.equal 15
|
|
|
|
assert.deepEqual(
|
|
|
|
results,
|
|
|
|
_.concat(
|
|
|
|
_.fill([1..10], 'Your email or password is incorrect. Please try again'),
|
|
|
|
_.fill([1..5], 'This account has had too many login requests. Please wait 2 minutes before trying to log in again')
|
|
|
|
)
|
|
|
|
)
|
|
|
|
done()
|
|
|
|
)
|
|
|
|
|
|
|
|
|
2016-09-27 10:55:51 +00:00
|
|
|
describe "LoginViaRegistration", ->
|
|
|
|
|
|
|
|
before (done) ->
|
|
|
|
@timeout(60000)
|
|
|
|
@user1 = new User()
|
|
|
|
@user2 = new User()
|
|
|
|
async.series [
|
|
|
|
(cb) => @user1.login cb
|
|
|
|
(cb) => @user1.logout cb
|
|
|
|
(cb) => redis.clearUserSessions @user1, cb
|
|
|
|
(cb) => @user2.login cb
|
|
|
|
(cb) => @user2.logout cb
|
|
|
|
(cb) => redis.clearUserSessions @user2, cb
|
|
|
|
], done
|
|
|
|
@project_id = null
|
|
|
|
|
|
|
|
describe "[Security] Trying to register/login as another user", ->
|
|
|
|
|
|
|
|
it 'should have user1 login', (done) ->
|
|
|
|
@user1.login (err) ->
|
|
|
|
expect(err?).to.equal false
|
|
|
|
done()
|
|
|
|
|
|
|
|
it 'should have user1 create a project', (done) ->
|
|
|
|
@user1.createProject 'Private Project', (err, project_id) =>
|
|
|
|
expect(err?).to.equal false
|
|
|
|
@project_id = project_id
|
|
|
|
done()
|
|
|
|
|
|
|
|
it 'should ensure user1 can access their project', (done) ->
|
|
|
|
expectProjectAccess @user1, @project_id, done
|
|
|
|
|
|
|
|
it 'should ensure user2 cannot access the project', (done) ->
|
|
|
|
expectNoProjectAccess @user2, @project_id, done
|
|
|
|
|
|
|
|
it 'should prevent user2 from login/register with user1 email address', (done) ->
|
|
|
|
tryLoginThroughRegistrationForm @user2, @user1.email, 'totally_not_the_right_password', (err, response, body) =>
|
|
|
|
expect(body.redir?).to.equal false
|
|
|
|
expect(body.message?).to.equal true
|
|
|
|
expect(body.message).to.have.all.keys('type', 'text')
|
|
|
|
expect(body.message.type).to.equal 'error'
|
|
|
|
done()
|
|
|
|
|
|
|
|
it 'should still ensure user2 cannot access the project', (done) ->
|
|
|
|
expectNoProjectAccess @user2, @project_id, done
|