| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280 |
- /* eslint-disable
- no-return-assign,
- */
- // TODO: This file was created by bulk-decaffeinate.
- // Fix any style issues and re-enable lint.
- /*
- * decaffeinate suggestions:
- * DS102: Remove unnecessary code created because of implicit returns
- * Full docs: https://github.com/decaffeinate/decaffeinate/blob/master/docs/suggestions.md
- */
- const { EventEmitter } = require('node:events')
- const { expect } = require('chai')
- const SandboxedModule = require('sandboxed-module')
- const modulePath = '../../../app/js/SessionSockets'
- const sinon = require('sinon')
- describe('SessionSockets', function () {
- beforeEach(function () {
- this.metrics = { inc: sinon.stub() }
- this.SessionSocketsModule = SandboxedModule.require(modulePath, {
- requires: {
- '@overleaf/metrics': this.metrics,
- },
- })
- this.io = new EventEmitter()
- this.id1 = Math.random().toString()
- this.id2 = Math.random().toString()
- const redisResponses = {
- error: [new Error('Redis: something went wrong'), null],
- unknownId: [null, null],
- }
- redisResponses[this.id1] = [null, { user: { _id: '123' } }]
- redisResponses[this.id2] = [null, { user: { _id: 'abc' } }]
- this.sessionStore = {
- get: sinon
- .stub()
- .callsFake((id, fn) => fn.apply(null, redisResponses[id])),
- }
- this.cookieParser = function (req, res, next) {
- req.signedCookies = req._signedCookies
- return next()
- }
- this.SessionSockets = this.SessionSocketsModule(
- this.io,
- this.sessionStore,
- this.cookieParser,
- 'ol.sid'
- )
- return (this.checkSocket = (socket, fn) => {
- this.SessionSockets.once('connection', fn)
- return this.io.emit('connection', socket)
- })
- })
- describe('without cookies', function () {
- beforeEach(function () {
- return (this.socket = { handshake: {} })
- })
- it('should return a lookup error', function (done) {
- return this.checkSocket(this.socket, error => {
- expect(error).to.exist
- expect(error.message).to.equal('could not look up session by key')
- return done()
- })
- })
- it('should not query redis', function (done) {
- return this.checkSocket(this.socket, () => {
- expect(this.sessionStore.get.called).to.equal(false)
- return done()
- })
- })
- it('should increment the session.cookie metric with status "none"', function (done) {
- return this.checkSocket(this.socket, () => {
- expect(this.metrics.inc).to.be.calledWith('session.cookie', 1, {
- status: 'none',
- })
- return done()
- })
- })
- })
- describe('with a different cookie', function () {
- beforeEach(function () {
- return (this.socket = { handshake: { _signedCookies: { other: 1 } } })
- })
- it('should return a lookup error', function (done) {
- return this.checkSocket(this.socket, error => {
- expect(error).to.exist
- expect(error.message).to.equal('could not look up session by key')
- return done()
- })
- })
- it('should not query redis', function (done) {
- return this.checkSocket(this.socket, () => {
- expect(this.sessionStore.get.called).to.equal(false)
- return done()
- })
- })
- })
- describe('with a cookie with an invalid signature', function () {
- beforeEach(function () {
- return (this.socket = {
- handshake: { _signedCookies: { 'ol.sid': false } },
- })
- })
- it('should return a lookup error', function (done) {
- return this.checkSocket(this.socket, error => {
- expect(error).to.exist
- expect(error.message).to.equal('could not look up session by key')
- return done()
- })
- })
- it('should not query redis', function (done) {
- return this.checkSocket(this.socket, () => {
- expect(this.sessionStore.get.called).to.equal(false)
- return done()
- })
- })
- it('should increment the session.cookie metric with status=bad-signature', function (done) {
- return this.checkSocket(this.socket, () => {
- expect(this.metrics.inc).to.be.calledWith('session.cookie', 1, {
- status: 'bad-signature',
- })
- return done()
- })
- })
- })
- describe('with a valid cookie and a failing session lookup', function () {
- beforeEach(function () {
- return (this.socket = {
- handshake: { _signedCookies: { 'ol.sid': 'error' } },
- })
- })
- it('should query redis', function (done) {
- return this.checkSocket(this.socket, () => {
- expect(this.sessionStore.get.called).to.equal(true)
- return done()
- })
- })
- it('should return a redis error', function (done) {
- return this.checkSocket(this.socket, error => {
- expect(error).to.exist
- expect(error.message).to.equal('Redis: something went wrong')
- return done()
- })
- })
- it('should increment the session.cookie metric with status=error', function (done) {
- return this.checkSocket(this.socket, () => {
- expect(this.metrics.inc).to.be.calledWith('session.cookie', 1, {
- status: 'error',
- })
- return done()
- })
- })
- })
- describe('with a valid cookie and no matching session', function () {
- beforeEach(function () {
- return (this.socket = {
- handshake: { _signedCookies: { 'ol.sid': 'unknownId' } },
- })
- })
- it('should query redis', function (done) {
- return this.checkSocket(this.socket, () => {
- expect(this.sessionStore.get.called).to.equal(true)
- return done()
- })
- })
- it('should return a lookup error', function (done) {
- return this.checkSocket(this.socket, error => {
- expect(error).to.exist
- expect(error.message).to.equal('could not look up session by key')
- return done()
- })
- })
- it('should increment the session.cookie metric with status=missing', function (done) {
- return this.checkSocket(this.socket, () => {
- expect(this.metrics.inc).to.be.calledWith('session.cookie', 1, {
- status: 'missing',
- })
- return done()
- })
- })
- })
- describe('with a valid cookie and a matching session', function () {
- beforeEach(function () {
- return (this.socket = {
- handshake: { _signedCookies: { 'ol.sid': this.id1 } },
- })
- })
- it('should query redis', function (done) {
- return this.checkSocket(this.socket, () => {
- expect(this.sessionStore.get.called).to.equal(true)
- return done()
- })
- })
- it('should not return an error', function (done) {
- return this.checkSocket(this.socket, error => {
- expect(error).to.not.exist
- return done()
- })
- })
- it('should return the session', function (done) {
- return this.checkSocket(this.socket, (error, s, session) => {
- if (error) return done(error)
- expect(session).to.deep.equal({ user: { _id: '123' } })
- return done()
- })
- })
- it('should increment the session.cookie metric with status=signed', function (done) {
- return this.checkSocket(this.socket, () => {
- expect(this.metrics.inc).to.be.calledWith('session.cookie', 1, {
- status: 'signed',
- })
- return done()
- })
- })
- })
- describe('with a different valid cookie and matching session', function () {
- beforeEach(function () {
- return (this.socket = {
- handshake: { _signedCookies: { 'ol.sid': this.id2 } },
- })
- })
- it('should query redis', function (done) {
- return this.checkSocket(this.socket, () => {
- expect(this.sessionStore.get.called).to.equal(true)
- return done()
- })
- })
- it('should not return an error', function (done) {
- return this.checkSocket(this.socket, error => {
- expect(error).to.not.exist
- return done()
- })
- })
- it('should return the other session', function (done) {
- return this.checkSocket(this.socket, (error, s, session) => {
- if (error) return done(error)
- expect(session).to.deep.equal({ user: { _id: 'abc' } })
- return done()
- })
- })
- it('should increment the session.cookie metric with status=error', function (done) {
- return this.checkSocket(this.socket, () => {
- expect(this.metrics.inc).to.be.calledWith('session.cookie', 1, {
- status: 'signed',
- })
- return done()
- })
- })
- })
- })
|