| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366 |
- const sinon = require('sinon')
- const { assert, expect } = require('chai')
- const modulePath = '../../../../app/src/Features/Compile/ClsiCookieManager.js'
- const SandboxedModule = require('sandboxed-module')
- const realRequst = require('request')
- describe('ClsiCookieManager', function () {
- beforeEach(function () {
- this.redis = {
- auth() {},
- get: sinon.stub(),
- setex: sinon.stub().callsArg(3),
- }
- this.project_id = '123423431321-proj-id'
- this.user_id = 'abc-user-id'
- this.request = {
- post: sinon.stub(),
- cookie: realRequst.cookie,
- jar: realRequst.jar,
- defaults: () => {
- return this.request
- },
- }
- this.settings = {
- redis: {
- web: 'redis.something',
- },
- apis: {
- clsi: {
- url: 'http://clsi.example.com',
- },
- },
- clsiCookie: {
- ttlInSeconds: Math.random().toString(),
- ttlInSecondsRegular: Math.random().toString(),
- key: 'coooookie',
- },
- }
- this.requires = {
- '../../infrastructure/RedisWrapper': (this.RedisWrapper = {
- client: () => this.redis,
- }),
- '@overleaf/settings': this.settings,
- request: this.request,
- }
- this.ClsiCookieManager = SandboxedModule.require(modulePath, {
- requires: this.requires,
- })()
- })
- describe('getServerId', function () {
- it('should call get for the key', function (done) {
- this.redis.get.callsArgWith(1, null, 'clsi-7')
- this.ClsiCookieManager.getServerId(
- this.project_id,
- this.user_id,
- '',
- 'e2',
- (err, serverId) => {
- if (err) {
- return done(err)
- }
- this.redis.get
- .calledWith(`clsiserver:${this.project_id}:${this.user_id}`)
- .should.equal(true)
- serverId.should.equal('clsi-7')
- done()
- }
- )
- })
- it('should _populateServerIdViaRequest if no key is found', function (done) {
- this.ClsiCookieManager._populateServerIdViaRequest = sinon
- .stub()
- .yields(null)
- this.redis.get.callsArgWith(1, null)
- this.ClsiCookieManager.getServerId(
- this.project_id,
- this.user_id,
- '',
- (err, serverId) => {
- if (err) {
- return done(err)
- }
- this.ClsiCookieManager._populateServerIdViaRequest
- .calledWith(this.project_id, this.user_id)
- .should.equal(true)
- done()
- }
- )
- })
- it('should _populateServerIdViaRequest if no key is blank', function (done) {
- this.ClsiCookieManager._populateServerIdViaRequest = sinon
- .stub()
- .yields(null)
- this.redis.get.callsArgWith(1, null, '')
- this.ClsiCookieManager.getServerId(
- this.project_id,
- this.user_id,
- '',
- 'e2',
- (err, serverId) => {
- if (err) {
- return done(err)
- }
- this.ClsiCookieManager._populateServerIdViaRequest
- .calledWith(this.project_id, this.user_id)
- .should.equal(true)
- done()
- }
- )
- })
- })
- describe('_populateServerIdViaRequest', function () {
- beforeEach(function () {
- this.clsiServerId = 'server-id'
- this.ClsiCookieManager.setServerId = sinon.stub().yields()
- })
- describe('with a server id in the response', function () {
- beforeEach(function () {
- this.response = {
- headers: {
- 'set-cookie': [
- `${this.settings.clsiCookie.key}=${this.clsiServerId}`,
- ],
- },
- }
- this.request.post.callsArgWith(1, null, this.response)
- })
- it('should make a request to the clsi', function (done) {
- this.ClsiCookieManager._populateServerIdViaRequest(
- this.project_id,
- this.user_id,
- 'standard',
- 'e2',
- (err, serverId) => {
- if (err) {
- return done(err)
- }
- const args = this.ClsiCookieManager.setServerId.args[0]
- args[0].should.equal(this.project_id)
- args[1].should.equal(this.user_id)
- args[2].should.equal('standard')
- args[3].should.equal('e2')
- args[4].should.deep.equal(this.clsiServerId)
- done()
- }
- )
- })
- it('should return the server id', function (done) {
- this.ClsiCookieManager._populateServerIdViaRequest(
- this.project_id,
- this.user_id,
- '',
- 'e2',
- (err, serverId) => {
- if (err) {
- return done(err)
- }
- serverId.should.equal(this.clsiServerId)
- done()
- }
- )
- })
- })
- describe('without a server id in the response', function () {
- beforeEach(function () {
- this.response = { headers: {} }
- this.request.post.yields(null, this.response)
- })
- it('should not set the server id there is no server id in the response', function (done) {
- this.ClsiCookieManager._parseServerIdFromResponse = sinon
- .stub()
- .returns(null)
- this.ClsiCookieManager.setServerId(
- this.project_id,
- this.user_id,
- 'standard',
- 'e2',
- this.clsiServerId,
- null,
- err => {
- if (err) {
- return done(err)
- }
- this.redis.setex.called.should.equal(false)
- done()
- }
- )
- })
- })
- })
- describe('setServerId', function () {
- beforeEach(function () {
- this.clsiServerId = 'server-id'
- this.ClsiCookieManager._parseServerIdFromResponse = sinon
- .stub()
- .returns('clsi-8')
- })
- it('should set the server id with a ttl', function (done) {
- this.ClsiCookieManager.setServerId(
- this.project_id,
- this.user_id,
- 'standard',
- 'e2',
- this.clsiServerId,
- null,
- err => {
- if (err) {
- return done(err)
- }
- this.redis.setex.should.have.been.calledWith(
- `clsiserver:${this.project_id}:${this.user_id}`,
- this.settings.clsiCookie.ttlInSeconds,
- this.clsiServerId
- )
- done()
- }
- )
- })
- it('should set the server id with the regular ttl for reg instance', function (done) {
- this.clsiServerId = 'clsi-reg-8'
- this.ClsiCookieManager.setServerId(
- this.project_id,
- this.user_id,
- 'standard',
- 'e2',
- this.clsiServerId,
- null,
- err => {
- if (err) {
- return done(err)
- }
- expect(this.redis.setex).to.have.been.calledWith(
- `clsiserver:${this.project_id}:${this.user_id}`,
- this.settings.clsiCookie.ttlInSecondsRegular,
- this.clsiServerId
- )
- done()
- }
- )
- })
- it('should not set the server id if clsiCookies are not enabled', function (done) {
- delete this.settings.clsiCookie.key
- this.ClsiCookieManager = SandboxedModule.require(modulePath, {
- globals: {
- console,
- },
- requires: this.requires,
- })()
- this.ClsiCookieManager.setServerId(
- this.project_id,
- this.user_id,
- 'standard',
- 'e2',
- this.clsiServerId,
- null,
- err => {
- if (err) {
- return done(err)
- }
- this.redis.setex.called.should.equal(false)
- done()
- }
- )
- })
- it('should also set in the secondary if secondary redis is enabled', function (done) {
- this.redis_secondary = { setex: sinon.stub().callsArg(3) }
- this.settings.redis.clsi_cookie_secondary = {}
- this.RedisWrapper.client = sinon.stub()
- this.RedisWrapper.client.withArgs('clsi_cookie').returns(this.redis)
- this.RedisWrapper.client
- .withArgs('clsi_cookie_secondary')
- .returns(this.redis_secondary)
- this.ClsiCookieManager = SandboxedModule.require(modulePath, {
- globals: {
- console,
- },
- requires: this.requires,
- })()
- this.ClsiCookieManager._parseServerIdFromResponse = sinon
- .stub()
- .returns('clsi-8')
- this.ClsiCookieManager.setServerId(
- this.project_id,
- this.user_id,
- 'standard',
- 'e2',
- this.clsiServerId,
- null,
- err => {
- if (err) {
- return done(err)
- }
- this.redis_secondary.setex.should.have.been.calledWith(
- `clsiserver:${this.project_id}:${this.user_id}`,
- this.settings.clsiCookie.ttlInSeconds,
- this.clsiServerId
- )
- done()
- }
- )
- })
- })
- describe('getCookieJar', function () {
- beforeEach(function () {
- this.ClsiCookieManager.getServerId = sinon.stub().yields(null, 'clsi-11')
- })
- it('should return a jar with the cookie set populated from redis', function (done) {
- this.ClsiCookieManager.getCookieJar(
- this.project_id,
- this.user_id,
- '',
- 'e2',
- (err, jar) => {
- if (err) {
- return done(err)
- }
- jar._jar.store.idx['clsi.example.com']['/'][
- this.settings.clsiCookie.key
- ].key.should.equal
- jar._jar.store.idx['clsi.example.com']['/'][
- this.settings.clsiCookie.key
- ].value.should.equal('clsi-11')
- done()
- }
- )
- })
- it('should return empty cookie jar if clsiCookies are not enabled', function (done) {
- delete this.settings.clsiCookie.key
- this.ClsiCookieManager = SandboxedModule.require(modulePath, {
- globals: {
- console,
- },
- requires: this.requires,
- })()
- this.ClsiCookieManager.getCookieJar(
- this.project_id,
- this.user_id,
- '',
- 'e2',
- (err, jar) => {
- if (err) {
- return done(err)
- }
- assert.deepEqual(jar, realRequst.jar())
- done()
- }
- )
- })
- })
- })
|