Commit Graph

42883 Commits

Author SHA1 Message Date
Gustavo Trott
736c9a130f refactor learning dashboard to use Reactions instead of Emojis 2024-08-26 13:40:17 -03:00
Paulo Lanzarin
8fb585162f
Merge pull request #20981 from bigbluebutton/docs-ms-ipv6-first
docs: change order of mediasoup IP config
2024-08-26 09:06:33 -03:00
João Victor Nunes
75b0b411f1
fix(presentation): poll results not appearing in the snapshot of slide (#21005) 2024-08-23 16:46:32 -03:00
Anton Barboza de Sá
6564659750
test: All param tests on ci (#20878)
* test: all param tests on ci

* test: remove check for notification on parameter test

* test: remove 'Force restore presentation on new poll result' test - keep only one using the parameter -, fix selector and add all necessary assertions for the cases when the presentation should be restored

* test: enable param test in CI

* test: update tests tagging
2024-08-23 14:32:58 -03:00
prlanzarin
dd1cf00dd2 feat: make guest policy optional for dial-in in akka-apps
Dial-in endpoints currently follow the same guest approval process as regular
users. While this is correct for default installations, it complicates the join
procedure in setups where approval is managed externally, such as with pre-
configured conference PIN codes. Another relevant scenario is when call
origination is done from BBB, rather than as an inbound call.

This commit makes the enforcement of the guest policy for dial-in/external
endpoints optional in akka-apps. The default behavior remains unchanged
(enforce). This provides more flexibility for the dial-in mechanism, allowing
for smoother handling of the scenarios mentioned above.
2024-08-23 14:54:31 +00:00
Ramón Souza
20dfac833d
fix connection status modal padding (#21009) 2024-08-23 11:04:45 -03:00
João Victor Nunes
8c5a0c8d25
fix(external-video): unsynched player when autoplay is blocked (#20991) 2024-08-23 10:16:15 -03:00
João Victor Nunes
fe67566334
[3.0] feat: Accept custom webcamBackgroundURL (#20920)
* [3.0] feat: Accept custom webcamBackgroundURL

* docs: webcamBackgroundURL

* test: Update virtual background thumbnail image

* fix: tweak error logs for the webcamBackgroundURL fetching procedure

* test: Update virtual background thumbnail image

* fix(logging): do not specify null extraInfo object

Co-authored-by: Paulo Lanzarin <4529051+prlanzarin@users.noreply.github.com>

* fix(logging): extract fields from error object for building extraInfo object

Co-authored-by: Paulo Lanzarin <4529051+prlanzarin@users.noreply.github.com>

---------

Co-authored-by: Paulo Lanzarin <4529051+prlanzarin@users.noreply.github.com>
2024-08-23 09:04:56 -03:00
prlanzarin
7fc8828c2b fix: audio settings UI adjustments
UI team suggested a few adjustments to the audio settings modal:
  - Larger (24px/1.5rem) margin between content and headers
  - Rephrasing of modal title, subtitle and volume indicator label
  - Change the "audio feedback" button to an outline or link styled
    button (there are currently two primary buttons and we want users to
    focus on the "Join audio" one)

Implement the suggested changes. The approach for the audio feedback
button is link-styled.
2024-08-23 03:05:10 +00:00
Gustavo Trott
f4749b0e96
Merge pull request #21003 from gustavotrott/webpack-fix
fix (html5-dev-server): Prevent HEAD requests from crashing webpack-dev-server
2024-08-22 18:17:18 -03:00
Átila
09f565d0ef
fix(darkmode): fix avatar username color in chat when dark mode is enabled (#20999) 2024-08-22 15:01:35 -03:00
Guilherme Leme
26ed9013e9 [fix-dom-element-manipulation] fix onClick argument 2024-08-22 11:29:35 -03:00
Guilherme Leme
bc6a6cbe9b [fix-dom-element-manipulation] Added new extensible area userCameraHelper 2024-08-22 10:51:00 -03:00
Gustavo Trott
e93388a6a2 prevent HEAD request from crashing webpack-dev-server 2024-08-22 10:44:08 -03:00
Paulo Lanzarin
f0738c8afb
Merge pull request #20997 from bigbluebutton/bbb30sfu2142
build(bbb-webrtc-sfu): v2.14.2
2024-08-21 20:28:08 -03:00
André Castro
635045ec84
Fix: Errors and warnings (#20994)
* The prop presentationIsOpen is marked as required in Presentation(null)

* The prop isPresentationManagementDisabled is marked as required in actions-dropdown(null)

* The prop autoJoin is marked as required in wake-lock(null)

* Al children must have key identifiers (userListParticipants)

* The prop presentationUploadExternalDescription is marked as required in presentation-uploadres
2024-08-21 17:22:54 -03:00
Paulo Lanzarin
3bdec18e17
build(bbb-webrtc-sfu): v2.14.2
v2.14.2
---
* refactor(audio): set FLOWING logs to INFO level
* build(mediasoup): v3.14.11
2024-08-21 16:16:30 -03:00
Átila
c48e93e00c
feat(darklogo): port customDarkLogo (#20922) 2024-08-21 14:58:32 -03:00
KDSBrowne
4ca3084db5
fix mouse wheel zoom not working on other pages (#20680) 2024-08-21 14:57:41 -03:00
Tainan Felipe
ba32e21c0f
Add: Local history of rtt status changes (#20983)
* Change: Add local history of rtt status changes
2024-08-21 10:05:37 -03:00
Paulo Lanzarin
80e71d619e
Merge pull request #20987 from bigbluebutton/sfu2141-msad
feat(audio): dedicate mediasoup workers for audio streams
2024-08-21 09:53:25 -03:00
Ramón Souza
07e79af097
new svg icon for reactions button (#20979) 2024-08-21 08:28:06 -03:00
Tainan Felipe
6597b41a8b
Add: store volume settings to preserve between player refresh (#20986) 2024-08-20 17:22:46 -03:00
prlanzarin
1e53140a00 fix(audio): prevent permission check loop in Safari
Safari may enter a microphone permission check loop due to buggy behavior
in the Permissions API. When permission isn't permanently denied, gUM
requests fail with a NotAllowedError for a few seconds. During this time,
the permission state remains 'prompt' instead of transitioning to 'denied'
and back to 'prompt' after the timeout.

This leads to an issue where, on retrying while in 'prompt' + blocked,
the client loops through gUM checks via: 1) checking permission in the API,
2) receiving 'prompt', so trying gUM, 3) gUM fails, 4) returning to the
modal and checking permission again because the API still says 'prompt'.

Additionally, the `isUsingAudio` flag incorrectly counts the local echo
test/audio settings modal as "using audio," which toggles the flag on/off,
triggering the useEffect that causes the loop more frequently.

To fix this, remove the unnecessary AudioModal permission check that
causes the loop. Also, exclude "isEchoTest" from the `isUsingAudio` flag.
2024-08-20 19:19:08 +00:00
Paulo Lanzarin
89209fc7ee
feat(audio): dedicate mediasoup workers for audio streams
mediasoup workers are currently for general use,
regardless of stream type. This makes it difficult to give
different scheduling priorities for audio workers or prevent
noise from video streams, when our goal is to give higher
priority to audio in all ends of the system.

Set `mediasoup.dedicatedMediaTypeWorkers.audio` to `auto`. This will spin up 
`ceil((min(nproc, 32) * 0.8) + (max(0, nproc - 32))` mediasoup workers 
dedicated to handling audio streams.
2024-08-20 15:06:32 -03:00
João Victor Nunes
3ad9aafd7b
fix: userdata-force_restore_presentation_on_new_events (#20974) 2024-08-20 13:29:04 -03:00
Paulo Lanzarin
a0fb450522
Merge pull request #20978 from prlanzarin/u30/fix/audio-ff-perm-api
fix(audio): ensure correct audio device labels in Firefox
2024-08-20 12:21:29 -03:00
Guilherme Leme
137af4e421 [fix-dom-element-manipulation] fix user-camera-dom-element-manipulation hook, refactored several hooks and added onclick event in dropdown 2024-08-20 10:42:05 -03:00
Paulo Lanzarin
c8dfa09fee
docs: change order of mediasoup IP config
See https://github.com/bigbluebutton/bigbluebutton/issues/20966.
The order of the configured mediasoup listenIps influence the server's candidate priorities. 
There's not a big impact to this, but if we want to be friendly to RFC 8421 (and to Happy Eyeballs), we should suggest sorting IPv6 first.

Change docs so that the IPv6 config in mediasoup comes first.
2024-08-20 10:41:52 -03:00
Paulo Lanzarin
1cd6edf9ac
Merge pull request #20977 from prlanzarin/u30/fix/did-output
fix(audio): ensure current I/O devices are flagged as selected
2024-08-20 09:45:42 -03:00
prlanzarin
b9f66c2a10 fix(audio): ensure correct audio device labels in Firefox
Firefox incorretly displays placeholder audio device labels in the audio
settings/echo test modal when audio is disconnected. This issue arises
due to two quirks:
- Firefox does not support the 'microphone' query from the Permissions
  API, causing a fallback gUM permission check.
- Firefox omits device labels from `enumerateDevices` if no streams
  are active, even if gUM permission is granted. This behavior differs
  from other browsers and causes our `enumerateDevices` handling to
  assume that granted permission implies labels are present. This
  failed since we clear streams before resolving the fallback gUM.

We now run an additional `enumerateDevices` call in `AudioSettings` when
a selected input device is defined. This ensures `enumerateDevices` is
re-run when a new stream is active, adding the correct device labels in
Firefox and improving device listings in all browsers. We've also
enhanced error handling in the enumeration process and fixed a false
positive in `hasMicrophonePermission`.
2024-08-20 00:43:24 +00:00
prlanzarin
c055caf915 fix(audio): ensure current I/O devices are flagged as selected
There's a regression in 3.0's I/O device selector where default output
devices are not marked as selected in the input-stream-live-selector
component unless the user explicitly selects them. This issue can also affect
input devices, although less commonly than output due to the system's ability
to infer the selected input device ID after the user joins audio.

When a device is the first in the list and no currentDeviceId is set in
the client, treat the first device returned by enumerateDevices as the
system default and hence selected, in accordance with the "Media Capture
and Streams API", Section 9.2, enumerateDevices algorithm.
2024-08-19 21:35:12 +00:00
Anton Barboza de Sá
c60ca2313b
test: Ask for feedback on logout test improvement + button display and action fixes (#20872)
* test: update and improve Ask for feedback on logout test - add more steps, check for different buttons, check POST request after sending feedback

* test: add missing data-test prop for sendFeendbackButton

* test: fix sendFeedbackButton data-test
2024-08-19 17:06:59 -03:00
Paulo Lanzarin
a2e2f28aba
Merge pull request #20963 from prlanzarin/u30/fix/tlo-f2
fix(audio): close dialog when "Cancel" is clicked with listenOnlyMode=false
2024-08-19 16:50:44 -03:00
Ramón Souza
91b5b7cb5f
Merge pull request #20937 from Tainan404/fix-connection-status
Fix: connection status issues
2024-08-19 16:24:05 -03:00
Ramón Souza
ddfba135a6
Merge pull request #20962 from Scroody/i-20817
Fix: Even number of emoji clicks hide who the sender was
2024-08-19 15:10:30 -03:00
Gustavo Trott
09cc37feef
refactor (gql-middleware): Code improvements and new Prometheus metrics (#20896) 2024-08-19 10:58:14 -03:00
Ramón Souza
af644be16b
Merge pull request #20953 from prlanzarin/u30/fix/plg-load-log
fix: use logger convention in plugin startup logs
2024-08-19 08:58:23 -03:00
Paulo Lanzarin
2e87bf2123
Merge pull request #20948 from prlanzarin/u30/fix/prescam-loadingstate
fix(screenshare): actions-bar loading state reacts to camera as content
2024-08-16 16:58:50 -03:00
André Castro
62fe6ed18c
Update component.jsx 2024-08-16 13:31:25 -03:00
prlanzarin
135350b6dc fix(audio): close dialog when "Cancel" is clicked with listenOnlyMode=false
When `listenOnlyMode` is `false` and the audio dialog's "Cancel" action is
clicked, the modal incorrectly re-renders instead of closing. Additionally,
the "Cancel" action is mislabeled as "Back."

This fix ensures the audio dialog closes properly when there are no options
to select (i.e., `listenOnlyMode=false`). The `skipAudioOptions` method is
revised to consider `listenOnlyMode` and ignore the "content" state.

Ignoring the "content" state allows options to be skipped even if a subscreen
is rendered (e.g., returning from the AudioSettings modal). The check for
`content == null` combined with `skipAudioOptions` is only necessary when
rendering the main modal. The `content == null` check has been moved to
the relevant section.
2024-08-16 15:48:47 +00:00
André Möller
6bbe3d4baa Fix: Even number of emoji clicks hide who the sender was 2024-08-16 11:52:40 -03:00
Ramón Souza
0a38236f94
Merge pull request #20873 from Arthurk12/presentation-last-state
fix(presentation): restore last state after sharing media
2024-08-16 10:48:11 -03:00
Ramón Souza
7c62352079
Merge pull request #20958 from JoVictorNunes/pure-voice-activity-hooks
fix: prevent the whole client from rerendering on voice activity
2024-08-16 09:47:57 -03:00
Paulo Lanzarin
5414372f8d
Merge pull request #20960 from prlanzarin/u30/fix/tlo-f1
fix(audio): prevent spurious mute toggle due to AudioSettings remount
2024-08-16 09:47:27 -03:00
prlanzarin
9040cb86cb fix(audio): prevent spurious mute toggle due to AudioSettings remount
When listen only mode is deactivated and an user joins audio, an incorrect
remount of AudioSettings can trigger a spurious mute toggle. This happens
because AudioManager clears the `isConnecting` flag before setting the
`isConnected` flag. This creates a brief period where audio is flagged as
"disconnected," leading to a remount and unmount cycle that causes unwanted
mute/unmute actions due to AudioSettings' logic of muting/unmuting
active devices.

Ensure the `isConnected` flag is set before clearing the
`isConnecting` flag, preventing audio from being incorrectly flagged as
disconnected.
2024-08-16 01:51:10 +00:00
Paulo Lanzarin
2639600e1d
Merge pull request #20957 from bigbluebutton/sfu2141
build(bbb-webrtc-sfu): v2.14.1
2024-08-15 17:04:53 -03:00
Paulo Lanzarin
a152cc45c5
build(bbb-webrtc-sfu): v2.14.1
v2.14.1
---
* fix(screenshare): log presenter/viewer stop on all scenarios
* refactor(screenshare): add presenter data to viewer logs
* refactor(video): add video negotiation and flowing logs
* build(mediasoup): 3.14.9
2024-08-15 16:42:48 -03:00
Ramón Souza
39d68e4156
Merge pull request #20782 from prlanzarin/u30/feat/tlo-ui-rebased250724
feat(audio): rework audio join UI for transparentListenOnly
2024-08-15 16:38:35 -03:00
Arthurk12
a9c8fd7051 fix(screenshare): removes redundant presentation restore
Removes redundant presentation restore code since the presentation restore
is executed by the layout context when updating the presentation pile.
2024-08-15 14:47:26 -03:00