Skip to content

Commit 3f14ee8

Browse files
authored
Fix publish permission check (#1420)
1 parent 75a245a commit 3f14ee8

File tree

1 file changed

+12
-8
lines changed

1 file changed

+12
-8
lines changed

src/room/participant/LocalParticipant.ts

Lines changed: 12 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -875,21 +875,25 @@ export default class LocalParticipant extends Participant {
875875

876876
private hasPermissionsToPublish(track: LocalTrack): boolean {
877877
if (!this.permissions) {
878+
this.log.warn('no permissions present for publishing track', {
879+
...this.logContext,
880+
...getLogContextFromTrack(track),
881+
});
878882
return false;
879883
}
880884
const { canPublish, canPublishSources } = this.permissions;
881885
if (
882-
!canPublish ||
886+
canPublish ||
883887
(canPublishSources &&
884-
!canPublishSources.map((source) => getTrackSourceFromProto(source)).includes(track.source))
888+
canPublishSources.map((source) => getTrackSourceFromProto(source)).includes(track.source))
885889
) {
886-
this.log.error('insufficient permissions to publish', {
887-
...this.logContext,
888-
...getLogContextFromTrack(track),
889-
});
890-
return false;
890+
return true;
891891
}
892-
return true;
892+
this.log.warn('insufficient permissions to publish', {
893+
...this.logContext,
894+
...getLogContextFromTrack(track),
895+
});
896+
return false;
893897
}
894898

895899
private async publish(track: LocalTrack, opts: TrackPublishOptions, isStereo: boolean) {

0 commit comments

Comments
 (0)