Fix AP security tests

pull/3888/head
Chocobozzz 2021-03-10 11:17:20 +01:00
parent 266131e0ca
commit e7053b1d9d
No known key found for this signature in database
GPG Key ID: 583A612D890159BE
5 changed files with 36 additions and 17 deletions

View File

@ -170,9 +170,11 @@ function buildGotOptions (options: PeerTubeRequestOptions) {
let headers = options.headers || {} let headers = options.headers || {}
headers = { ...headers, date: new Date().toUTCString() } if (!headers.date) {
headers = { ...headers, date: new Date().toUTCString() }
}
if (activityPub) { if (activityPub && !headers.accept) {
headers = { ...headers, accept: ACTIVITY_PUB.ACCEPT_HEADER } headers = { ...headers, accept: ACTIVITY_PUB.ACCEPT_HEADER }
} }

View File

@ -46,9 +46,9 @@ async function buildSignedRequestOptions (payload: Payload<any>) {
function buildGlobalHeaders (body: any) { function buildGlobalHeaders (body: any) {
return { return {
'Digest': buildDigest(body), 'digest': buildDigest(body),
'Content-Type': 'application/activity+json', 'content-type': 'application/activity+json',
'Accept': ACTIVITY_PUB.ACCEPT_HEADER 'accept': ACTIVITY_PUB.ACCEPT_HEADER
} }
} }

View File

@ -23,7 +23,7 @@ const signatureValidator = [
.custom(isSignatureValueValid).withMessage('Should have a valid signature value'), .custom(isSignatureValueValid).withMessage('Should have a valid signature value'),
(req: express.Request, res: express.Response, next: express.NextFunction) => { (req: express.Request, res: express.Response, next: express.NextFunction) => {
logger.debug('Checking activitypub signature parameter', { parameters: { signature: req.body.signature } }) logger.debug('Checking Linked Data Signature parameter', { parameters: { signature: req.body.signature } })
if (areValidationErrors(req, res)) return if (areValidationErrors(req, res)) return

View File

@ -8,6 +8,8 @@ import {
cleanupTests, cleanupTests,
closeAllSequelize, closeAllSequelize,
flushAndRunMultipleServers, flushAndRunMultipleServers,
killallServers,
reRunServer,
ServerInfo, ServerInfo,
setActorField, setActorField,
wait wait
@ -20,21 +22,32 @@ import { buildGlobalHeaders } from '../../../lib/job-queue/handlers/utils/activi
const expect = chai.expect const expect = chai.expect
function setKeysOfServer (onServer: ServerInfo, ofServer: ServerInfo, publicKey: string, privateKey: string) { function setKeysOfServer (onServer: ServerInfo, ofServer: ServerInfo, publicKey: string, privateKey: string) {
const url = 'http://localhost:' + ofServer.port + '/accounts/peertube'
return Promise.all([ return Promise.all([
setActorField(onServer.internalServerNumber, 'http://localhost:' + ofServer.port + '/accounts/peertube', 'publicKey', publicKey), setActorField(onServer.internalServerNumber, url, 'publicKey', publicKey),
setActorField(onServer.internalServerNumber, 'http://localhost:' + ofServer.port + '/accounts/peertube', 'privateKey', privateKey) setActorField(onServer.internalServerNumber, url, 'privateKey', privateKey)
]) ])
} }
function getAnnounceWithoutContext (server2: ServerInfo) { function setUpdatedAtOfServer (onServer: ServerInfo, ofServer: ServerInfo, updatedAt: string) {
const url = 'http://localhost:' + ofServer.port + '/accounts/peertube'
return Promise.all([
setActorField(onServer.internalServerNumber, url, 'createdAt', updatedAt),
setActorField(onServer.internalServerNumber, url, 'updatedAt', updatedAt)
])
}
function getAnnounceWithoutContext (server: ServerInfo) {
const json = require('./json/peertube/announce-without-context.json') const json = require('./json/peertube/announce-without-context.json')
const result: typeof json = {} const result: typeof json = {}
for (const key of Object.keys(json)) { for (const key of Object.keys(json)) {
if (Array.isArray(json[key])) { if (Array.isArray(json[key])) {
result[key] = json[key].map(v => v.replace(':9002', `:${server2.port}`)) result[key] = json[key].map(v => v.replace(':9002', `:${server.port}`))
} else { } else {
result[key] = json[key].replace(':9002', `:${server2.port}`) result[key] = json[key].replace(':9002', `:${server.port}`)
} }
} }
@ -64,7 +77,8 @@ describe('Test ActivityPub security', function () {
url = servers[0].url + '/inbox' url = servers[0].url + '/inbox'
await setKeysOfServer(servers[0], servers[1], keys.publicKey, keys.privateKey) await setKeysOfServer(servers[0], servers[1], keys.publicKey, null)
await setKeysOfServer(servers[1], servers[1], keys.publicKey, keys.privateKey)
const to = { url: 'http://localhost:' + servers[0].port + '/accounts/peertube' } const to = { url: 'http://localhost:' + servers[0].port + '/accounts/peertube' }
const by = { url: 'http://localhost:' + servers[1].port + '/accounts/peertube', privateKey: keys.privateKey } const by = { url: 'http://localhost:' + servers[1].port + '/accounts/peertube', privateKey: keys.privateKey }
@ -152,12 +166,14 @@ describe('Test ActivityPub security', function () {
it('Should refresh the actor keys', async function () { it('Should refresh the actor keys', async function () {
this.timeout(20000) this.timeout(20000)
// Wait refresh invalidation
await wait(10000)
// Update keys of server 2 to invalid keys // Update keys of server 2 to invalid keys
// Server 1 should refresh the actor and fail // Server 1 should refresh the actor and fail
await setKeysOfServer(servers[1], servers[1], invalidKeys.publicKey, invalidKeys.privateKey) await setKeysOfServer(servers[1], servers[1], invalidKeys.publicKey, invalidKeys.privateKey)
await setUpdatedAtOfServer(servers[0], servers[1], '2015-07-17 22:00:00+00')
// Invalid peertube actor cache
killallServers([ servers[1] ])
await reRunServer(servers[1])
const body = activityPubContextify(getAnnounceWithoutContext(servers[1])) const body = activityPubContextify(getAnnounceWithoutContext(servers[1]))
const headers = buildGlobalHeaders(body) const headers = buildGlobalHeaders(body)
@ -166,6 +182,7 @@ describe('Test ActivityPub security', function () {
await makePOSTAPRequest(url, body, baseHttpSignature(), headers) await makePOSTAPRequest(url, body, baseHttpSignature(), headers)
expect(true, 'Did not throw').to.be.false expect(true, 'Did not throw').to.be.false
} catch (err) { } catch (err) {
console.error(err)
expect(err.statusCode).to.equal(HttpStatusCode.FORBIDDEN_403) expect(err.statusCode).to.equal(HttpStatusCode.FORBIDDEN_403)
} }
}) })

View File

@ -17,7 +17,7 @@ function makePOSTAPRequest (url: string, body: any, httpSignature: any, headers:
async function makeFollowRequest (to: { url: string }, by: { url: string, privateKey }) { async function makeFollowRequest (to: { url: string }, by: { url: string, privateKey }) {
const follow = { const follow = {
type: 'Follow', type: 'Follow',
id: by.url + '/toto', id: by.url + '/' + new Date().getTime(),
actor: by.url, actor: by.url,
object: to.url object: to.url
} }
@ -33,7 +33,7 @@ async function makeFollowRequest (to: { url: string }, by: { url: string, privat
} }
const headers = buildGlobalHeaders(body) const headers = buildGlobalHeaders(body)
return makePOSTAPRequest(to.url, body, httpSignature, headers) return makePOSTAPRequest(to.url + '/inbox', body, httpSignature, headers)
} }
export { export {