github/overleaf
1
0
Fork 0
mirror of https://github.com/overleaf/overleaf.git synced 2024-09-16 02:52:31 -04:00
Code Issues Projects Releases Packages Wiki Activity

Merge pull request #15473 from overleaf/jel-remove-encoded-hash

Browse source 
[web] Remove encoded hashtag when checking token hash prefix

GitOrigin-RevId: 2b9c369beceb7897d0f6f165b4c3a70b79c14469
This commit is contained in:
Jessica Lawshe 2023-10-30 08:52:12 -05:00 committed by Copybot
parent 3cd0cefd30
commit a3251394e1
2 changed files with 19 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
services/web/app/src/Features/TokenAccess/TokenAccessHandler.js
View file

@ -291,7 +291,7 @@ const TokenAccessHandler = {
let hashPrefixStatus
if (tokenHashPrefix) {
tokenHashPrefix = tokenHashPrefix.replace('#', '')
tokenHashPrefix = tokenHashPrefix.replace('#', '').replace('%23', '')
}
if (!tokenHashPrefix) {

18
services/web/test/unit/src/TokenAccess/TokenAccessHandlerTests.js
View file

@ -719,5 +719,23 @@ describe('TokenAccessHandler', function () {
}
)
})
it('it handles encoded hashtags', function () {
const token = 'zxpxjrwdtsgd'
const prefix = this.TokenAccessHandler.createTokenHashPrefix(token)
this.TokenAccessHandler.checkTokenHashPrefix(
token,
`%23${prefix}`,
'readOnly'
)
expect(this.Metrics.inc).to.have.been.calledWith(
'link-sharing.hash-check',
{
path: 'readOnly',
status: 'match',
}
)
})
})
})