OpenCloud
/
PeerTube
mirror of https://github.com/Chocobozzz/PeerTube
1
0
Fork 0
Code Issues Releases Wiki Activity

Fix playlist get for classic users

pull/1856/head
Chocobozzz 2019-05-28 09:50:27 +02:00
parent ec893ae075
commit 4d09cfba78
No known key found for this signature in database
GPG Key ID: 583A612D890159BE
2 changed files with 16 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
server/middlewares/validators/videos/video-playlists.ts
View File

@ -140,9 +140,10 @@ const videoPlaylistsGetValidator = [
await authenticatePromiseIfNeeded(req, res)
const user = res.locals.oauth ? res.locals.oauth.token.User : null
if (
!user ||
(videoPlaylist.OwnerAccount.userId !== user.id && !user.hasRight(UserRight.UPDATE_ANY_VIDEO_PLAYLIST))
(videoPlaylist.OwnerAccount.id !== user.Account.id && !user.hasRight(UserRight.UPDATE_ANY_VIDEO_PLAYLIST))
) {
return res.status(403)
.json({ error: 'Cannot get this private video playlist.' })

15
server/tests/api/videos/video-playlists.ts
View File

@ -36,7 +36,8 @@ import {
uploadVideo,
uploadVideoAndGetId,
userLogin,
waitJobs
waitJobs,
generateUserAccessToken
} from '../../../../shared/extra-utils'
import { VideoPlaylistPrivacy } from '../../../../shared/models/videos/playlist/video-playlist-privacy.model'
import { VideoPlaylist } from '../../../../shared/models/videos/playlist/video-playlist.model'
@ -138,6 +139,18 @@ describe('Test video playlists', function () {
}
})
it('Should get private playlist for a classic user', async function () {
const token = await generateUserAccessToken(servers[0], 'toto')
const res = await getAccountPlaylistsListWithToken(servers[0].url, token, 'toto', 0, 5)
expect(res.body.total).to.equal(1)
expect(res.body.data).to.have.lengthOf(1)
const playlistId = res.body.data[0].id
await getPlaylistVideos(servers[0].url, token, playlistId, 0, 5)
})
it('Should create a playlist on server 1 and have the playlist on server 2 and 3', async function () {
this.timeout(30000)