| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129 |
- const SandboxedModule = require('sandboxed-module')
- const sinon = require('sinon')
- const modulePath = require('path').join(
- __dirname,
- '../../../../app/src/Features/Security/RateLimiterMiddleware'
- )
- describe('RateLimiterMiddleware', function () {
- beforeEach(function () {
- this.SessionManager = {
- getLoggedInUserId: () => this.req.session?.user?._id,
- }
- this.RateLimiterMiddleware = SandboxedModule.require(modulePath, {
- requires: {
- '@overleaf/settings': (this.settings = {}),
- './LoginRateLimiter': {},
- '../Authentication/SessionManager': this.SessionManager,
- },
- })
- this.req = { params: {} }
- this.res = {
- status: sinon.stub(),
- write: sinon.stub(),
- end: sinon.stub(),
- }
- this.next = sinon.stub()
- })
- describe('rateLimit', function () {
- beforeEach(function () {
- this.projectId = 'project-id'
- this.docId = 'doc-id'
- this.rateLimiter = {
- consume: sinon.stub().resolves({ remainingPoints: 2 }),
- }
- this.middleware = this.RateLimiterMiddleware.rateLimit(this.rateLimiter, {
- params: ['projectId', 'docId'],
- })
- this.req.params = { projectId: this.projectId, docId: this.docId }
- })
- describe('when there is no session', function () {
- beforeEach(function (done) {
- this.req.ip = this.ip = '1.2.3.4'
- this.middleware(this.req, this.res, () => {
- done()
- })
- })
- it('should call the rate limiter with the ip address', function () {
- this.rateLimiter.consume.should.have.been.calledWith(
- `${this.projectId}:${this.docId}:${this.ip}`
- )
- })
- })
- describe('when smoke test user', function () {
- beforeEach(function (done) {
- this.userId = 'smoke-test-user-id'
- this.req.session = {
- user: { _id: this.userId },
- }
- this.settings.smokeTest = { userId: this.userId }
- this.middleware(this.req, this.res, () => {
- done()
- })
- })
- it('should not call the rate limiter', function () {
- this.rateLimiter.consume.should.not.have.been.called
- })
- })
- describe('when under the rate limit with logged in user', function () {
- beforeEach(function (done) {
- this.userId = 'user-id'
- this.req.session = {
- user: { _id: this.userId },
- }
- this.middleware(this.req, this.res, () => {
- done()
- })
- })
- it('should call the rate limiter backend with the userId', function () {
- this.rateLimiter.consume.should.have.been.calledWith(
- `${this.projectId}:${this.docId}:${this.userId}`
- )
- })
- })
- describe('when under the rate limit with anonymous user', function () {
- beforeEach(function (done) {
- this.req.ip = '1.2.3.4'
- this.middleware(this.req, this.res, () => {
- done()
- })
- })
- it('should call the rate limiter backend with the ip address', function () {
- this.rateLimiter.consume.should.have.been.calledWith(
- `${this.projectId}:${this.docId}:${this.req.ip}`
- )
- })
- })
- describe('when over the rate limit', function () {
- beforeEach(function (done) {
- this.userId = 'user-id'
- this.req.session = {
- user: { _id: this.userId },
- }
- this.res.end.callsFake(() => {
- done()
- })
- this.rateLimiter.consume.rejects({ remainingPoints: 0 })
- this.middleware(this.req, this.res, this.next)
- })
- it('should return a 429', function () {
- this.res.status.should.have.been.calledWith(429)
- })
- it('should not continue', function () {
- this.next.should.not.have.been.called
- })
- })
- })
- })
|