| 12345678910111213141516171819202122232425262728293031323334353637383940414243444546474849505152535455565758596061626364656667686970717273747576777879808182838485868788899091929394959697989910010110210310410510610710810911011111211311411511611711811912012112212312412512612712812913013113213313413513613713813914014114214314414514614714814915015115215315415515615715815916016116216316416516616716816917017117217317417517617717817918018118218318418518618718818919019119219319419519619719819920020120220320420520620720820921021121221321421521621721821922022122222322422522622722822923023123223323423523623723823924024124224324424524624724824925025125225325425525625725825926026126226326426526626726826927027127227327427527627727827928028128228328428528628728828929029129229329429529629729829930030130230330430530630730830931031131231331431531631731831932032132232332432532632732832933033133233333433533633733833934034134234334434534634734834935035135235335435535635735835936036136236336436536636736836937037137237337437537637737837938038138238338438538638738838939039139239339439539639739839940040140240340440540640740840941041141241341441541641741841942042142242342442542642742842943043143243343443543643743843944044144244344444544644744844945045145245345445545645745845946046146246346446546646746846947047147247347447547647747847948048148248348448548648748848949049149249349449549649749849950050150250350450550650750850951051151251351451551651751851952052152252352452552652752852953053153253353453553653753853954054154254354454554654754854955055155255355455555655755855956056156256356456556656756856957057157257357457557657757857958058158258358458558658758858959059159259359459559659759859960060160260360460560660760860961061161261361461561661761861962062162262362462562662762862963063163263363463563663763863964064164264364464564664764864965065165265365465565665765865966066166266366466566666766866967067167267367467567667767867968068168268368468568668768868969069169269369469569669769869970070170270370470570670770870971071171271371471571671771871972072172272372472572672772872973073173273373473573673773873974074174274374474574674774874975075175275375475575675775875976076176276376476576676776876977077177277377477577677777877978078178278378478578678778878979079179279379479579679779879980080180280380480580680780880981081181281381481581681781881982082182282382482582682782882983083183283383483583683783883984084184284384484584684784884985085185285385485585685785885986086186286386486586686786886987087187287387487587687787887988088188288388488588688788888989089189289389489589689789889990090190290390490590690790890991091191291391491591691791891992092192292392492592692792892993093193293393493593693793893994094194294394494594694794894995095195295395495595695795895996096196296396496596696796896997097197297397497597697797897998098198298398498598698798898999099199299399499599699799899910001001100210031004100510061007100810091010101110121013101410151016101710181019102010211022102310241025102610271028102910301031103210331034103510361037103810391040104110421043104410451046104710481049105010511052105310541055105610571058105910601061106210631064106510661067106810691070107110721073107410751076107710781079108010811082108310841085108610871088108910901091109210931094109510961097109810991100110111021103110411051106110711081109111011111112111311141115111611171118111911201121112211231124112511261127112811291130113111321133113411351136113711381139114011411142114311441145114611471148114911501151115211531154115511561157115811591160116111621163116411651166116711681169117011711172117311741175117611771178117911801181118211831184118511861187118811891190119111921193119411951196119711981199120012011202120312041205120612071208120912101211121212131214121512161217121812191220122112221223122412251226122712281229123012311232123312341235123612371238123912401241124212431244124512461247124812491250125112521253125412551256125712581259126012611262126312641265126612671268126912701271127212731274127512761277 |
- import { expect, vi } from 'vitest'
- import sinon from 'sinon'
- import mongodb from 'mongodb-legacy'
- import MockRequest from '../helpers/MockRequest.js'
- import MockResponse from '../helpers/MockResponse.js'
- import PrivilegeLevels from '../../../../app/src/Features/Authorization/PrivilegeLevels.js'
- import { getSafeRedirectPath } from '../../../../app/src/Features/Helpers/UrlHelper.js'
- const ObjectId = mongodb.ObjectId
- const MODULE_PATH =
- '../../../../app/src/Features/TokenAccess/TokenAccessController'
- describe('TokenAccessController', function () {
- beforeEach(async function (ctx) {
- ctx.token = 'abc123'
- ctx.user = { _id: new ObjectId() }
- ctx.project = {
- _id: new ObjectId(),
- owner_ref: ctx.user._id,
- name: 'test',
- tokenAccessReadAndWrite_refs: [],
- tokenAccessReadOnly_refs: [],
- }
- ctx.req = new MockRequest()
- ctx.res = new MockResponse()
- ctx.next = sinon.stub().returns()
- ctx.Settings = {
- siteUrl: 'https://www.dev-overleaf.com',
- adminPrivilegeAvailable: false,
- adminUrl: 'https://admin.dev-overleaf.com',
- adminDomains: ['overleaf.com'],
- }
- ctx.TokenAccessHandler = {
- TOKEN_TYPES: {
- READ_ONLY: 'readOnly',
- READ_AND_WRITE: 'readAndWrite',
- },
- isReadAndWriteToken: sinon.stub().returns(true),
- isReadOnlyToken: sinon.stub().returns(true),
- tokenAccessEnabledForProject: sinon.stub().returns(true),
- checkTokenHashPrefix: sinon.stub(),
- makeTokenUrl: sinon.stub().returns('/'),
- grantSessionTokenAccess: sinon.stub(),
- promises: {
- addReadOnlyUserToProject: sinon.stub().resolves(),
- getProjectByToken: sinon.stub().resolves(ctx.project),
- getV1DocPublishedInfo: sinon.stub().resolves({ allow: true }),
- getV1DocInfo: sinon.stub(),
- removeReadAndWriteUserFromProject: sinon.stub().resolves(),
- moveReadAndWriteUserToReadOnly: sinon.stub().resolves(),
- },
- }
- ctx.SessionManager = {
- getLoggedInUserId: sinon.stub().returns(ctx.user._id),
- getSessionUser: sinon.stub().returns(ctx.user._id),
- }
- ctx.AuthenticationController = {
- setRedirectInSession: sinon.stub(),
- }
- ctx.AuthorizationManager = {
- promises: {
- getPrivilegeLevelForProject: sinon
- .stub()
- .resolves(PrivilegeLevels.NONE),
- },
- }
- ctx.AuthorizationMiddleware = {}
- ctx.ProjectAuditLogHandler = {
- promises: {
- addEntry: sinon.stub().resolves(),
- },
- }
- ctx.SplitTestHandler = {
- promises: {
- getAssignment: sinon.stub().resolves({ variant: 'default' }),
- getAssignmentForUser: sinon.stub().resolves({ variant: 'default' }),
- },
- }
- ctx.CollaboratorsInviteHandler = {
- promises: {
- revokeInviteForUser: sinon.stub().resolves(),
- },
- }
- ctx.CollaboratorsHandler = {
- promises: {
- addUserIdToProject: sinon.stub().resolves(),
- setCollaboratorPrivilegeLevel: sinon.stub().resolves(),
- },
- }
- ctx.CollaboratorsGetter = {
- promises: {
- userIsReadWriteTokenMember: sinon.stub().resolves(),
- isUserInvitedReadWriteMemberOfProject: sinon.stub().resolves(),
- isUserInvitedMemberOfProject: sinon.stub().resolves(),
- },
- }
- ctx.EditorRealTimeController = { emitToRoom: sinon.stub() }
- ctx.ProjectGetter = {
- promises: {
- getProject: sinon.stub().resolves(ctx.project),
- },
- }
- ctx.AnalyticsManager = {
- recordEventForSession: sinon.stub(),
- recordEventForUserInBackground: sinon.stub(),
- }
- ctx.UserGetter = {
- promises: {
- getUser: sinon.stub().callsFake(async (userId, filter) => {
- if (userId === ctx.userId) {
- return ctx.user
- } else {
- return null
- }
- }),
- getUserEmail: sinon.stub().resolves(),
- getUserConfirmedEmails: sinon.stub().resolves(),
- },
- }
- ctx.LimitationsManager = {
- promises: {
- canAcceptEditCollaboratorInvite: sinon.stub().resolves(),
- },
- }
- vi.doMock('@overleaf/settings', () => ({
- default: ctx.Settings,
- }))
- vi.doMock(
- '../../../../app/src/Features/TokenAccess/TokenAccessHandler',
- () => ({
- default: ctx.TokenAccessHandler,
- })
- )
- vi.doMock(
- '../../../../app/src/Features/Authentication/AuthenticationController',
- () => ({
- default: ctx.AuthenticationController,
- })
- )
- vi.doMock(
- '../../../../app/src/Features/Authentication/SessionManager',
- () => ({
- default: ctx.SessionManager,
- })
- )
- vi.doMock(
- '../../../../app/src/Features/Authorization/AuthorizationManager',
- () => ({
- default: ctx.AuthorizationManager,
- })
- )
- vi.doMock(
- '../../../../app/src/Features/Authorization/AuthorizationMiddleware',
- () => ({
- default: ctx.AuthorizationMiddleware,
- })
- )
- vi.doMock(
- '../../../../app/src/Features/Project/ProjectAuditLogHandler',
- () => ({
- default: ctx.ProjectAuditLogHandler,
- })
- )
- vi.doMock(
- '../../../../app/src/Features/SplitTests/SplitTestHandler',
- () => ({
- default: ctx.SplitTestHandler,
- })
- )
- vi.doMock('../../../../app/src/Features/Errors/Errors', () => ({
- default: (ctx.Errors = {
- NotFoundError: sinon.stub(),
- }),
- }))
- vi.doMock(
- '../../../../app/src/Features/Collaborators/CollaboratorsHandler',
- () => ({
- default: ctx.CollaboratorsHandler,
- })
- )
- vi.doMock(
- '../../../../app/src/Features/Collaborators/CollaboratorsInviteHandler',
- () => ({
- default: ctx.CollaboratorsInviteHandler,
- })
- )
- vi.doMock(
- '../../../../app/src/Features/Collaborators/CollaboratorsGetter',
- () => ({
- default: ctx.CollaboratorsGetter,
- })
- )
- vi.doMock(
- '../../../../app/src/Features/Editor/EditorRealTimeController',
- () => ({
- default: ctx.EditorRealTimeController,
- })
- )
- vi.doMock('../../../../app/src/Features/Project/ProjectGetter', () => ({
- default: ctx.ProjectGetter,
- }))
- vi.doMock('../../../../app/src/Features/Helpers/AsyncFormHelper', () => ({
- default: (ctx.AsyncFormHelper = {
- redirect: sinon.stub(),
- }),
- }))
- vi.doMock(
- '../../../../app/src/Features/Helpers/AdminAuthorizationHelper',
- () =>
- (ctx.AdminAuthorizationHelper = {
- canRedirectToAdminDomain: sinon.stub(),
- })
- )
- vi.doMock(
- '../../../../app/src/Features/Helpers/UrlHelper',
- () =>
- (ctx.UrlHelper = {
- getSafeAdminDomainRedirect: sinon
- .stub()
- .callsFake(
- path => `${ctx.Settings.adminUrl}${getSafeRedirectPath(path)}`
- ),
- })
- )
- vi.doMock(
- '../../../../app/src/Features/Analytics/AnalyticsManager',
- () => ({
- default: ctx.AnalyticsManager,
- })
- )
- vi.doMock('../../../../app/src/Features/User/UserGetter', () => ({
- default: ctx.UserGetter,
- }))
- vi.doMock(
- '../../../../app/src/Features/Subscription/LimitationsManager',
- () => ({
- default: ctx.LimitationsManager,
- })
- )
- ctx.TokenAccessController = (await import(MODULE_PATH)).default
- })
- describe('grantTokenAccessReadAndWrite', function () {
- beforeEach(function (ctx) {
- ctx.LimitationsManager.promises.canAcceptEditCollaboratorInvite.resolves(
- true
- )
- })
- describe('normal case (edit slot available)', function () {
- beforeEach(async function (ctx) {
- await new Promise(resolve => {
- ctx.LimitationsManager.promises.canAcceptEditCollaboratorInvite.resolves(
- true
- )
- ctx.req.params = { token: ctx.token }
- ctx.req.body = {
- confirmedByUser: true,
- tokenHashPrefix: '#prefix',
- }
- ctx.res.callback = resolve
- ctx.TokenAccessController.grantTokenAccessReadAndWrite(
- ctx.req,
- ctx.res,
- resolve
- )
- })
- })
- it('adds the user as a read and write invited member', function (ctx) {
- expect(
- ctx.CollaboratorsHandler.promises.addUserIdToProject
- ).to.have.been.calledWith(
- ctx.project._id,
- undefined,
- ctx.user._id,
- PrivilegeLevels.READ_AND_WRITE
- )
- })
- it('writes a project audit log', function (ctx) {
- expect(
- ctx.ProjectAuditLogHandler.promises.addEntry
- ).to.have.been.calledWith(
- ctx.project._id,
- 'accept-via-link-sharing',
- ctx.user._id,
- ctx.req.ip,
- { privileges: 'readAndWrite' }
- )
- })
- it('records a project-joined event for the user', function (ctx) {
- expect(
- ctx.AnalyticsManager.recordEventForUserInBackground
- ).to.have.been.calledWith(ctx.user._id, 'project-joined', {
- mode: 'edit',
- projectId: ctx.project._id.toString(),
- ownerId: ctx.project.owner_ref.toString(),
- role: PrivilegeLevels.READ_AND_WRITE,
- source: 'link-sharing',
- })
- })
- it('emits a project membership changed event', function (ctx) {
- expect(ctx.EditorRealTimeController.emitToRoom).to.have.been.calledWith(
- ctx.project._id,
- 'project:membership:changed',
- { members: true, invites: true }
- )
- })
- it('checks token hash', function (ctx) {
- expect(
- ctx.TokenAccessHandler.checkTokenHashPrefix
- ).to.have.been.calledWith(
- ctx.token,
- '#prefix',
- 'readAndWrite',
- ctx.user._id,
- { projectId: ctx.project._id, action: 'continue' }
- )
- })
- })
- describe('when there are no edit collaborator slots available', function () {
- beforeEach(async function (ctx) {
- await new Promise(resolve => {
- ctx.LimitationsManager.promises.canAcceptEditCollaboratorInvite.resolves(
- false
- )
- ctx.req.params = { token: ctx.token }
- ctx.req.body = {
- confirmedByUser: true,
- tokenHashPrefix: '#prefix',
- }
- ctx.res.callback = resolve
- ctx.TokenAccessController.grantTokenAccessReadAndWrite(
- ctx.req,
- ctx.res,
- resolve
- )
- })
- })
- it('adds the user as a read only invited member instead (pendingEditor)', function (ctx) {
- expect(
- ctx.CollaboratorsHandler.promises.addUserIdToProject
- ).to.have.been.calledWith(
- ctx.project._id,
- undefined,
- ctx.user._id,
- PrivilegeLevels.READ_ONLY,
- { pendingEditor: true }
- )
- })
- it('writes a project audit log', function (ctx) {
- expect(
- ctx.ProjectAuditLogHandler.promises.addEntry
- ).to.have.been.calledWith(
- ctx.project._id,
- 'accept-via-link-sharing',
- ctx.user._id,
- ctx.req.ip,
- { privileges: 'readOnly', pendingEditor: true }
- )
- })
- it('records a project-joined event for the user', function (ctx) {
- expect(
- ctx.AnalyticsManager.recordEventForUserInBackground
- ).to.have.been.calledWith(ctx.user._id, 'project-joined', {
- mode: 'view',
- projectId: ctx.project._id.toString(),
- pendingEditor: true,
- ownerId: ctx.project.owner_ref.toString(),
- role: PrivilegeLevels.READ_ONLY,
- source: 'link-sharing',
- })
- })
- it('emits a project membership changed event', function (ctx) {
- expect(ctx.EditorRealTimeController.emitToRoom).to.have.been.calledWith(
- ctx.project._id,
- 'project:membership:changed',
- { members: true, invites: true }
- )
- })
- it('checks token hash', function (ctx) {
- expect(
- ctx.TokenAccessHandler.checkTokenHashPrefix
- ).to.have.been.calledWith(
- ctx.token,
- '#prefix',
- 'readAndWrite',
- ctx.user._id,
- { projectId: ctx.project._id, action: 'continue' }
- )
- })
- })
- describe('when the access was already granted', function () {
- beforeEach(async function (ctx) {
- await new Promise(resolve => {
- ctx.project.tokenAccessReadAndWrite_refs.push(ctx.user._id)
- ctx.req.params = { token: ctx.token }
- ctx.req.body = { confirmedByUser: true }
- ctx.res.callback = resolve
- ctx.TokenAccessController.grantTokenAccessReadAndWrite(
- ctx.req,
- ctx.res,
- resolve
- )
- })
- })
- it('writes a project audit log', function (ctx) {
- expect(
- ctx.ProjectAuditLogHandler.promises.addEntry
- ).to.have.been.calledWith(
- ctx.project._id,
- 'accept-via-link-sharing',
- ctx.user._id,
- ctx.req.ip,
- { privileges: 'readAndWrite' }
- )
- })
- it('checks token hash', function (ctx) {
- expect(
- ctx.TokenAccessHandler.checkTokenHashPrefix
- ).to.have.been.calledWith(
- ctx.token,
- undefined,
- 'readAndWrite',
- ctx.user._id,
- { projectId: ctx.project._id, action: 'continue' }
- )
- })
- })
- describe('hash prefix missing in request', function () {
- beforeEach(async function (ctx) {
- await new Promise(resolve => {
- ctx.req.params = { token: ctx.token }
- ctx.req.body = { confirmedByUser: true }
- ctx.res.callback = resolve
- ctx.TokenAccessController.grantTokenAccessReadAndWrite(
- ctx.req,
- ctx.res,
- resolve
- )
- })
- })
- it('adds the user as a read and write invited member', function (ctx) {
- expect(
- ctx.CollaboratorsHandler.promises.addUserIdToProject
- ).to.have.been.calledWith(
- ctx.project._id,
- undefined,
- ctx.user._id,
- PrivilegeLevels.READ_AND_WRITE
- )
- })
- it('checks the hash prefix', function (ctx) {
- expect(
- ctx.TokenAccessHandler.checkTokenHashPrefix
- ).to.have.been.calledWith(
- ctx.token,
- undefined,
- 'readAndWrite',
- ctx.user._id,
- { projectId: ctx.project._id, action: 'continue' }
- )
- })
- })
- describe('user is owner of project', function () {
- beforeEach(async function (ctx) {
- await new Promise(resolve => {
- ctx.AuthorizationManager.promises.getPrivilegeLevelForProject.returns(
- PrivilegeLevels.OWNER
- )
- ctx.req.params = { token: ctx.token }
- ctx.req.body = {}
- ctx.res.callback = resolve
- ctx.TokenAccessController.grantTokenAccessReadAndWrite(
- ctx.req,
- ctx.res,
- resolve
- )
- })
- })
- it('checks token hash and includes log data', function (ctx) {
- expect(
- ctx.TokenAccessHandler.checkTokenHashPrefix
- ).to.have.been.calledWith(
- ctx.token,
- undefined,
- 'readAndWrite',
- ctx.user._id,
- {
- projectId: ctx.project._id,
- action: 'user already has higher or same privilege',
- }
- )
- })
- })
- describe('when user is not logged in', function () {
- beforeEach(function (ctx) {
- ctx.SessionManager.getLoggedInUserId.returns(null)
- ctx.req.params = { token: ctx.token }
- ctx.req.body = { tokenHashPrefix: '#prefix' }
- })
- describe('ANONYMOUS_READ_AND_WRITE_ENABLED is undefined', function () {
- beforeEach(async function (ctx) {
- await new Promise(resolve => {
- ctx.res.callback = resolve
- ctx.TokenAccessController.grantTokenAccessReadAndWrite(
- ctx.req,
- ctx.res,
- resolve
- )
- })
- })
- it('redirects to restricted', function (ctx) {
- expect(ctx.res.json).to.have.been.calledWith({
- redirect: '/restricted',
- anonWriteAccessDenied: true,
- })
- })
- it('checks the hash prefix and includes log data', function (ctx) {
- expect(
- ctx.TokenAccessHandler.checkTokenHashPrefix
- ).to.have.been.calledWith(
- ctx.token,
- '#prefix',
- 'readAndWrite',
- null,
- {
- action: 'denied anonymous read-and-write token access',
- }
- )
- })
- it('saves redirect URL with URL fragment', function (ctx) {
- expect(
- ctx.AuthenticationController.setRedirectInSession.lastCall.args[1]
- ).to.equal('/#prefix')
- })
- })
- describe('ANONYMOUS_READ_AND_WRITE_ENABLED is true', function () {
- beforeEach(async function (ctx) {
- await new Promise(resolve => {
- ctx.TokenAccessHandler.ANONYMOUS_READ_AND_WRITE_ENABLED = true
- ctx.res.callback = resolve
- ctx.TokenAccessController.grantTokenAccessReadAndWrite(
- ctx.req,
- ctx.res,
- resolve
- )
- })
- })
- it('redirects to project', function (ctx) {
- expect(ctx.res.json).to.have.been.calledWith({
- redirect: `/project/${ctx.project._id}`,
- grantAnonymousAccess: 'readAndWrite',
- })
- })
- it('checks the hash prefix and includes log data', function (ctx) {
- expect(
- ctx.TokenAccessHandler.checkTokenHashPrefix
- ).to.have.been.calledWith(
- ctx.token,
- '#prefix',
- 'readAndWrite',
- null,
- {
- projectId: ctx.project._id,
- action: 'granting read-write anonymous access',
- }
- )
- })
- })
- })
- describe('when Overleaf SaaS', function () {
- beforeEach(function (ctx) {
- ctx.Settings.overleaf = {}
- })
- describe('when token is for v1 project', function () {
- beforeEach(async function (ctx) {
- await new Promise(resolve => {
- ctx.TokenAccessHandler.promises.getProjectByToken.resolves(
- undefined
- )
- ctx.TokenAccessHandler.promises.getV1DocInfo.resolves({
- exists: true,
- has_owner: true,
- })
- ctx.req.params = { token: ctx.token }
- ctx.req.body = { tokenHashPrefix: '#prefix' }
- ctx.res.callback = resolve
- ctx.TokenAccessController.grantTokenAccessReadAndWrite(
- ctx.req,
- ctx.res,
- resolve
- )
- })
- })
- it('returns v1 import data', function (ctx) {
- expect(ctx.res.json).to.have.been.calledWith({
- v1Import: {
- status: 'canDownloadZip',
- projectId: ctx.token,
- hasOwner: true,
- name: 'Untitled',
- brandInfo: undefined,
- },
- })
- })
- it('checks the hash prefix and includes log data', function (ctx) {
- expect(
- ctx.TokenAccessHandler.checkTokenHashPrefix
- ).to.have.been.calledWith(
- ctx.token,
- '#prefix',
- 'readAndWrite',
- ctx.user._id,
- {
- action: 'import v1',
- }
- )
- })
- })
- describe('when token is not for a v1 or v2 project', function () {
- beforeEach(async function (ctx) {
- await new Promise(resolve => {
- ctx.TokenAccessHandler.promises.getProjectByToken.resolves(
- undefined
- )
- ctx.TokenAccessHandler.promises.getV1DocInfo.resolves({
- exists: false,
- })
- ctx.req.params = { token: ctx.token }
- ctx.req.body = { tokenHashPrefix: '#prefix' }
- ctx.res.callback = resolve
- ctx.TokenAccessController.grantTokenAccessReadAndWrite(
- ctx.req,
- ctx.res,
- resolve
- )
- })
- })
- it('returns 404', function (ctx) {
- expect(ctx.res.sendStatus).to.have.been.calledWith(404)
- })
- it('checks the hash prefix and includes log data', function (ctx) {
- expect(
- ctx.TokenAccessHandler.checkTokenHashPrefix
- ).to.have.been.calledWith(
- ctx.token,
- '#prefix',
- 'readAndWrite',
- ctx.user._id,
- {
- action: '404',
- }
- )
- })
- })
- })
- describe('not Overleaf SaaS', function () {
- beforeEach(function (ctx) {
- ctx.TokenAccessHandler.promises.getProjectByToken.resolves(undefined)
- ctx.req.params = { token: ctx.token }
- ctx.req.body = { tokenHashPrefix: '#prefix' }
- })
- it('passes Errors.NotFoundError to next when project not found and still checks token hash', async function (ctx) {
- await new Promise(resolve => {
- ctx.TokenAccessController.grantTokenAccessReadAndWrite(
- ctx.req,
- ctx.res,
- args => {
- expect(args).to.be.instanceof(ctx.Errors.NotFoundError)
- expect(
- ctx.TokenAccessHandler.checkTokenHashPrefix
- ).to.have.been.calledWith(
- ctx.token,
- '#prefix',
- 'readAndWrite',
- ctx.user._id,
- {
- action: '404',
- }
- )
- resolve()
- }
- )
- })
- })
- })
- describe('when user is admin', function () {
- const admin = { _id: new ObjectId(), isAdmin: true }
- beforeEach(function (ctx) {
- ctx.SessionManager.getLoggedInUserId.returns(admin._id)
- ctx.SessionManager.getSessionUser.returns(admin)
- ctx.AdminAuthorizationHelper.canRedirectToAdminDomain.returns(true)
- ctx.req.params = { token: ctx.token }
- ctx.req.body = { confirmedByUser: true, tokenHashPrefix: '#prefix' }
- })
- it('redirects if project owner is non-admin', async function (ctx) {
- ctx.UserGetter.promises.getUserConfirmedEmails = sinon
- .stub()
- .resolves([{ email: 'test@not-overleaf.com' }])
- await new Promise(resolve => {
- ctx.res.callback = () => {
- expect(ctx.res.json).to.have.been.calledWith({
- redirect: `${ctx.Settings.adminUrl}/#prefix`,
- })
- resolve()
- }
- ctx.TokenAccessController.grantTokenAccessReadAndWrite(
- ctx.req,
- ctx.res
- )
- })
- })
- it('grants access if project owner is an internal staff', function (ctx) {
- const internalStaff = { _id: new ObjectId(), isAdmin: true }
- const projectFromInternalStaff = {
- _id: new ObjectId(),
- name: 'test',
- tokenAccessReadAndWrite_refs: [],
- tokenAccessReadOnly_refs: [],
- owner_ref: internalStaff._id,
- }
- ctx.UserGetter.promises.getUser = sinon.stub().resolves(internalStaff)
- ctx.UserGetter.promises.getUserConfirmedEmails = sinon
- .stub()
- .resolves([{ email: 'test@overleaf.com' }])
- ctx.TokenAccessHandler.promises.getProjectByToken = sinon
- .stub()
- .resolves(projectFromInternalStaff)
- ctx.res.callback = () => {
- expect(
- ctx.CollaboratorsHandler.promises.addUserIdToProject
- ).to.have.been.calledWith(
- projectFromInternalStaff._id,
- undefined,
- admin._id,
- PrivilegeLevels.READ_AND_WRITE
- )
- }
- ctx.TokenAccessController.grantTokenAccessReadAndWrite(ctx.req, ctx.res)
- })
- })
- it('passes Errors.NotFoundError to next when token access is not enabled but still checks token hash', async function (ctx) {
- await new Promise(resolve => {
- ctx.TokenAccessHandler.tokenAccessEnabledForProject.returns(false)
- ctx.req.params = { token: ctx.token }
- ctx.req.body = { tokenHashPrefix: '#prefix' }
- ctx.TokenAccessController.grantTokenAccessReadAndWrite(
- ctx.req,
- ctx.res,
- args => {
- expect(args).to.be.instanceof(ctx.Errors.NotFoundError)
- expect(
- ctx.TokenAccessHandler.checkTokenHashPrefix
- ).to.have.been.calledWith(
- ctx.token,
- '#prefix',
- 'readAndWrite',
- ctx.user._id,
- {
- projectId: ctx.project._id,
- action: 'token access not enabled',
- }
- )
- resolve()
- }
- )
- })
- })
- it('returns 400 when not using a read write token', function (ctx) {
- ctx.TokenAccessHandler.isReadAndWriteToken.returns(false)
- ctx.req.params = { token: ctx.token }
- ctx.req.body = { tokenHashPrefix: '#prefix' }
- ctx.TokenAccessController.grantTokenAccessReadAndWrite(ctx.req, ctx.res)
- expect(ctx.res.sendStatus).to.have.been.calledWith(400)
- })
- })
- describe('grantTokenAccessReadOnly', function () {
- describe('normal case', function () {
- beforeEach(async function (ctx) {
- await new Promise(resolve => {
- ctx.req.params = { token: ctx.token }
- ctx.req.body = { confirmedByUser: true, tokenHashPrefix: '#prefix' }
- ctx.res.callback = resolve
- ctx.TokenAccessController.grantTokenAccessReadOnly(
- ctx.req,
- ctx.res,
- resolve
- )
- })
- })
- it('grants read-only access', function (ctx) {
- expect(
- ctx.TokenAccessHandler.promises.addReadOnlyUserToProject
- ).to.have.been.calledWith(
- ctx.user._id,
- ctx.project._id,
- ctx.project.owner_ref
- )
- })
- it('writes a project audit log', function (ctx) {
- expect(
- ctx.ProjectAuditLogHandler.promises.addEntry
- ).to.have.been.calledWith(
- ctx.project._id,
- 'join-via-token',
- ctx.user._id,
- ctx.req.ip,
- { privileges: 'readOnly' }
- )
- })
- it('checks if hash prefix matches', function (ctx) {
- expect(
- ctx.TokenAccessHandler.checkTokenHashPrefix
- ).to.have.been.calledWith(
- ctx.token,
- '#prefix',
- 'readOnly',
- ctx.user._id,
- { projectId: ctx.project._id, action: 'continue' }
- )
- })
- })
- describe('when the access was already granted', function () {
- beforeEach(async function (ctx) {
- await new Promise(resolve => {
- ctx.project.tokenAccessReadOnly_refs.push(ctx.user._id)
- ctx.req.params = { token: ctx.token }
- ctx.req.body = { confirmedByUser: true }
- ctx.res.callback = resolve
- ctx.TokenAccessController.grantTokenAccessReadOnly(
- ctx.req,
- ctx.res,
- resolve
- )
- })
- })
- it("doesn't write a project audit log", function (ctx) {
- expect(ctx.ProjectAuditLogHandler.promises.addEntry).to.not.have.been
- .called
- })
- it('still checks if hash prefix matches', function (ctx) {
- expect(
- ctx.TokenAccessHandler.checkTokenHashPrefix
- ).to.have.been.calledWith(
- ctx.token,
- undefined,
- 'readOnly',
- ctx.user._id,
- { projectId: ctx.project._id, action: 'continue' }
- )
- })
- })
- it('returns 400 when not using a read only token', function (ctx) {
- ctx.TokenAccessHandler.isReadOnlyToken.returns(false)
- ctx.req.params = { token: ctx.token }
- ctx.req.body = { tokenHashPrefix: '#prefix' }
- ctx.TokenAccessController.grantTokenAccessReadOnly(ctx.req, ctx.res)
- expect(ctx.res.sendStatus).to.have.been.calledWith(400)
- })
- describe('anonymous users', function () {
- beforeEach(async function (ctx) {
- await new Promise(resolve => {
- ctx.req.params = { token: ctx.token }
- ctx.SessionManager.getLoggedInUserId.returns(null)
- ctx.res.callback = resolve
- ctx.TokenAccessController.grantTokenAccessReadOnly(
- ctx.req,
- ctx.res,
- resolve
- )
- })
- })
- it('allows anonymous users and checks the token hash', function (ctx) {
- expect(ctx.res.json).to.have.been.calledWith({
- redirect: `/project/${ctx.project._id}`,
- grantAnonymousAccess: 'readOnly',
- })
- expect(
- ctx.TokenAccessHandler.checkTokenHashPrefix
- ).to.have.been.calledWith(ctx.token, undefined, 'readOnly', null, {
- projectId: ctx.project._id,
- action: 'granting read-only anonymous access',
- })
- })
- })
- describe('user is owner of project', function () {
- beforeEach(async function (ctx) {
- await new Promise(resolve => {
- ctx.AuthorizationManager.promises.getPrivilegeLevelForProject.returns(
- PrivilegeLevels.OWNER
- )
- ctx.req.params = { token: ctx.token }
- ctx.req.body = {}
- ctx.res.callback = resolve
- ctx.TokenAccessController.grantTokenAccessReadOnly(
- ctx.req,
- ctx.res,
- resolve
- )
- })
- })
- it('checks token hash and includes log data', function (ctx) {
- expect(
- ctx.TokenAccessHandler.checkTokenHashPrefix
- ).to.have.been.calledWith(
- ctx.token,
- undefined,
- 'readOnly',
- ctx.user._id,
- {
- projectId: ctx.project._id,
- action: 'user already has higher or same privilege',
- }
- )
- })
- })
- it('passes Errors.NotFoundError to next when token access is not enabled but still checks token hash', async function (ctx) {
- await new Promise(resolve => {
- ctx.TokenAccessHandler.tokenAccessEnabledForProject.returns(false)
- ctx.req.params = { token: ctx.token }
- ctx.req.body = { tokenHashPrefix: '#prefix' }
- ctx.TokenAccessController.grantTokenAccessReadOnly(
- ctx.req,
- ctx.res,
- args => {
- expect(args).to.be.instanceof(ctx.Errors.NotFoundError)
- expect(
- ctx.TokenAccessHandler.checkTokenHashPrefix
- ).to.have.been.calledWith(
- ctx.token,
- '#prefix',
- 'readOnly',
- ctx.user._id,
- {
- projectId: ctx.project._id,
- action: 'token access not enabled',
- }
- )
- resolve()
- }
- )
- })
- })
- })
- describe('ensureUserCanUseSharingUpdatesConsentPage', function () {
- beforeEach(function (ctx) {
- ctx.req.params = { Project_id: ctx.project._id }
- })
- describe('when not in link sharing changes test', function () {
- beforeEach(async function (ctx) {
- await new Promise(resolve => {
- ctx.AsyncFormHelper.redirect = sinon.stub().callsFake(() => resolve())
- ctx.TokenAccessController.ensureUserCanUseSharingUpdatesConsentPage(
- ctx.req,
- ctx.res,
- resolve
- )
- })
- })
- it('redirects to the project/editor', function (ctx) {
- expect(ctx.AsyncFormHelper.redirect).to.have.been.calledWith(
- ctx.req,
- ctx.res,
- `/project/${ctx.project._id}`
- )
- })
- })
- describe('when link sharing changes test active', function () {
- beforeEach(function (ctx) {
- ctx.SplitTestHandler.promises.getAssignmentForUser.resolves({
- variant: 'active',
- })
- })
- describe('when user is not an invited editor and is a read write token member', function () {
- beforeEach(async function (ctx) {
- await new Promise(resolve => {
- ctx.CollaboratorsGetter.promises.isUserInvitedReadWriteMemberOfProject.resolves(
- false
- )
- ctx.CollaboratorsGetter.promises.userIsReadWriteTokenMember.resolves(
- true
- )
- ctx.next.callsFake(() => resolve())
- ctx.TokenAccessController.ensureUserCanUseSharingUpdatesConsentPage(
- ctx.req,
- ctx.res,
- ctx.next
- )
- })
- })
- it('calls next', function (ctx) {
- expect(
- ctx.CollaboratorsGetter.promises
- .isUserInvitedReadWriteMemberOfProject
- ).to.have.been.calledWith(ctx.user._id, ctx.project._id)
- expect(
- ctx.CollaboratorsGetter.promises.userIsReadWriteTokenMember
- ).to.have.been.calledWith(ctx.user._id, ctx.project._id)
- expect(ctx.next).to.have.been.calledOnce
- expect(ctx.next.firstCall.args[0]).to.not.exist
- })
- })
- describe('when user is already an invited editor', function () {
- beforeEach(async function (ctx) {
- await new Promise(resolve => {
- ctx.CollaboratorsGetter.promises.isUserInvitedReadWriteMemberOfProject.resolves(
- true
- )
- ctx.AsyncFormHelper.redirect = sinon
- .stub()
- .callsFake(() => resolve())
- ctx.TokenAccessController.ensureUserCanUseSharingUpdatesConsentPage(
- ctx.req,
- ctx.res,
- resolve
- )
- })
- })
- it('redirects to the project/editor', function (ctx) {
- expect(ctx.AsyncFormHelper.redirect).to.have.been.calledWith(
- ctx.req,
- ctx.res,
- `/project/${ctx.project._id}`
- )
- })
- })
- describe('when user not a read write token member', function () {
- beforeEach(async function (ctx) {
- await new Promise(resolve => {
- ctx.CollaboratorsGetter.promises.userIsReadWriteTokenMember.resolves(
- false
- )
- ctx.AsyncFormHelper.redirect = sinon
- .stub()
- .callsFake(() => resolve())
- ctx.TokenAccessController.ensureUserCanUseSharingUpdatesConsentPage(
- ctx.req,
- ctx.res,
- resolve
- )
- })
- })
- it('redirects to the project/editor', function (ctx) {
- expect(ctx.AsyncFormHelper.redirect).to.have.been.calledWith(
- ctx.req,
- ctx.res,
- `/project/${ctx.project._id}`
- )
- })
- })
- })
- })
- describe('moveReadWriteToCollaborators', function () {
- beforeEach(function (ctx) {
- ctx.req.params = { Project_id: ctx.project._id }
- })
- describe('when there are collaborator slots available', function () {
- beforeEach(function (ctx) {
- ctx.LimitationsManager.promises.canAcceptEditCollaboratorInvite.resolves(
- true
- )
- })
- describe('previously joined token access user moving to named collaborator', function () {
- beforeEach(async function (ctx) {
- await new Promise(resolve => {
- ctx.CollaboratorsGetter.promises.isUserInvitedMemberOfProject.resolves(
- false
- )
- ctx.res.callback = resolve
- ctx.TokenAccessController.moveReadWriteToCollaborators(
- ctx.req,
- ctx.res,
- resolve
- )
- })
- })
- it('sets the privilege level to read and write for the invited viewer', function (ctx) {
- expect(
- ctx.TokenAccessHandler.promises.removeReadAndWriteUserFromProject
- ).to.have.been.calledWith(ctx.user._id, ctx.project._id)
- expect(
- ctx.CollaboratorsHandler.promises.addUserIdToProject
- ).to.have.been.calledWith(
- ctx.project._id,
- undefined,
- ctx.user._id,
- PrivilegeLevels.READ_AND_WRITE
- )
- expect(ctx.res.sendStatus).to.have.been.calledWith(204)
- })
- })
- })
- describe('when there are no edit collaborator slots available', function () {
- beforeEach(function (ctx) {
- ctx.LimitationsManager.promises.canAcceptEditCollaboratorInvite.resolves(
- false
- )
- })
- describe('previously joined token access user moving to named collaborator', function () {
- beforeEach(async function (ctx) {
- await new Promise(resolve => {
- ctx.CollaboratorsGetter.promises.isUserInvitedMemberOfProject.resolves(
- false
- )
- ctx.res.callback = resolve
- ctx.TokenAccessController.moveReadWriteToCollaborators(
- ctx.req,
- ctx.res,
- resolve
- )
- })
- })
- it('sets the privilege level to read only for the invited viewer (pendingEditor)', function (ctx) {
- expect(
- ctx.TokenAccessHandler.promises.removeReadAndWriteUserFromProject
- ).to.have.been.calledWith(ctx.user._id, ctx.project._id)
- expect(
- ctx.CollaboratorsHandler.promises.addUserIdToProject
- ).to.have.been.calledWith(
- ctx.project._id,
- undefined,
- ctx.user._id,
- PrivilegeLevels.READ_ONLY,
- { pendingEditor: true }
- )
- expect(ctx.res.sendStatus).to.have.been.calledWith(204)
- })
- })
- })
- })
- describe('moveReadWriteToReadOnly', function () {
- beforeEach(function (ctx) {
- ctx.req.params = { Project_id: ctx.project._id }
- })
- describe('previously joined token access user moving to anonymous viewer', function () {
- beforeEach(async function (ctx) {
- await new Promise(resolve => {
- ctx.res.callback = resolve
- ctx.TokenAccessController.moveReadWriteToReadOnly(
- ctx.req,
- ctx.res,
- resolve
- )
- })
- })
- it('removes them from read write token access refs and adds them to read only token access refs', function (ctx) {
- expect(
- ctx.TokenAccessHandler.promises.moveReadAndWriteUserToReadOnly
- ).to.have.been.calledWith(ctx.user._id, ctx.project._id)
- expect(ctx.res.sendStatus).to.have.been.calledWith(204)
- })
- it('writes a project audit log', function (ctx) {
- expect(
- ctx.ProjectAuditLogHandler.promises.addEntry
- ).to.have.been.calledWith(
- ctx.project._id,
- 'readonly-via-sharing-updates',
- ctx.user._id,
- ctx.req.ip
- )
- })
- })
- })
- })
|