fix(permissions): adds project owner to stream perms

makes sure #118 doesn't happen fixes #118
speckleworks · Dec 4, 2019 · 51ddf13 · 51ddf13
1 parent 779751b
commit 51ddf13
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/app/api/projects/ProjectPutAddStream.js b/app/api/projects/ProjectPutAddStream.js
@@ -20,10 +20,15 @@ module.exports = async ( req, res ) => {
     project.permissions.canRead.forEach( id => {
       stream.canRead.indexOf( id ) === -1 ? stream.canRead.push( id ) : null
     } )
+
+    stream.canRead.indexOf( project.owner ) === -1 ? stream.canRead.push( project.owner ) : null
+
     project.permissions.canWrite.forEach( id => {
       stream.canWrite.indexOf( id ) === -1 ? stream.canWrite.push( id ) : null
     } )
 
+    stream.canWrite.indexOf( project.owner ) === -1 ? stream.canWrite.push( project.owner ) : null
+
     await Promise.all( [ stream.save( ), project.save( ) ] )
     return res.send( { success: true, project: project, stream: stream } )
   } catch ( err ) {