nextcloud / nextcloud/spreed

Missing display name during calls from participants with denied perms

Open
#4,454 6 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bug feature: call 📹
Dominant language
PHP
Stars
2.2k
Forks
587
Avg merge
17h 55m
Merged PRs (30d)
350

Description

Steps to reproduce

  1. Create three users, all start with the same letter and make sure to set their display names
  2. Have a call with all those
  3. Try and distinguish the avatars in the call view
Expected behaviour

Display name should be visible somewhere

Actual behaviour

Display name missing.

The code shows that display name was intended to be there, but the source of the name is null:
image

Talk app

Talk app version: 10.0.0 with NC 20.0.0

Custom Signaling server configured: yes

Custom TURN server configured: no

Custom STUN server configured: no

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reproducing the call view with three participants whose display names share an initial, following the steps in the issue. Inspect the participant display-name source shown in the attached screenshot, especially when permissions are denied; done means the display name is visible somewhere in the call view.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
audio-video-rtc, frontend
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
38/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.