Commit Graph

9109 Commits

Author SHA1 Message Date
Robert Adam
58fee9705d CHANGE(server): Untrimmed usernames considered invalid
When checking whether a given username is valid, it being trimmed is now
also a criterion. All names with leading and/or trailing whitespace will
be considered invalid from now on.
2021-04-11 13:24:41 +02:00
Robert Adam
90190bcb01 CHANGE(server): Always trim usernames
A username will now always be trimmed before it gets assigned to a user.
That means that names with leading and/or trailing whitespace are now no
longer possible.
2021-04-11 13:23:12 +02:00
Davide Beatrici
da5b6edd07
Merge PR #4918: MAINT: Add Code of Conduct 2021-04-10 04:25:42 +02:00
Robert Adam
f04d6635a9
Merge PR #4920: FIX(client): Check the returned error code of ALSA
The functions snd_card_get_name and snd_card_get_longname allocate a
String and return 0 if success otherwise the negative error code.
There is a need to check the return value before using the allocated String.

This commit fixes this by checking the return value and exiting if the
return value is not zero.

Ref:alsa-project.org/alsa-doc/alsa-lib/group___control.html#gad48d44da8e3bfe150e928267008b8ff5
2021-04-08 18:21:40 +02:00
Robert Adam
ae94142c51 TRANSLATION: Update translation files 2021-04-08 11:39:30 +02:00
Zero0one1
6632dabd91 FIX(client): Check the returned error code of ALSA
The functions snd_card_get_name and snd_card_get_longname allocate a
String and return 0 if success otherwise the negative error code.
There is a need to check the return value before using the allocated String.

This commit fixes this by checking the return value and exiting if the
return value is not zero.

Ref: #4910

Co-Authored-By: Zero0one1 <summerlinasity@gmail.com>
Co-Authored-By: Krzmbrzl <krzmbrzl@gmail.com>
2021-04-08 11:39:18 +02:00
Robert Adam
e8f5c7037b
Merge PR #4915: FIX(server): Fix undefined behavior on Linux hosts
The fields of the rlimit struct were assumed to be of type long int when
printing via the qWarning function. This seems to be correct on e.g.
Ubuntu but at least on Alpine Linux this is not true.

In that case the wrong format specifiers are being used for printing
these values, which causes undefined behavior.

The fix is to always copy the values into variables of type long long
unsigned int and then printing these. The code also adds an assertion
making sure that within this copy operation the values won't overflow
the variable's size.

Fixes #4897
2021-04-08 11:33:53 +02:00
Robert Adam
cd190fd313
Merge PR #4919: MAINT: Update hash in Ice port file (vcpkg)
The cmake-based fork (mumble-voip/ice) recently has
received a new commit making changes that obviously also changed the SHA
checksum when installing this Ice version via vcpkg.

The mentioned commit is
mumble-voip/ice@a9d68a8

This commit therefore updates the mentioned hash in the corresponding
portfile.
2021-04-08 09:19:42 +02:00
Davide Beatrici
012f8af725 MAINT: Add Code of Conduct
Co-authored-by: Robert Adam <dev@robert-adam.de>
2021-04-07 19:58:21 +02:00
DRC
636063b2c6 MAINT: Update hash in Ice port file (vcpkg)
The cmake-based fork (https://github.com/mumble-voip/ice/) recently has
received a new commit making changes that obviously also changed the SHA
checksum when installing this Ice version via vcpkg.

The mentioned commit is
a9d68a8d1d

This commit therefore updates the mentioned hash in the corresponding
portfile.
2021-04-07 16:21:03 +02:00
Robert Adam
6e0ab7a6ae
Merge PR #4916: FIX(client): Memory leak in ALSA implementation
The functions snd_card_get_name and snd_card_get_longname allocate a
String for the caller that needs to be freed once it is no longer needed.
This was not done in the current implementation and thus this was causing
a memory leak.

This commit fixes this by explicitly freeing the resources after usage.

Fixes #4910

Ref: alsa-project.org/alsa-doc/alsa-lib/group___control.html#ga547dcead4a72f24db4a0f9f530f75228
2021-04-06 11:52:39 +02:00
Zero0one1
7c5188a577 FIX(client): Memory leak in ALSA implementation
The functions snd_card_get_name and snd_card_get_longname allocate a
String for the caller that needs to be freed once it is no longer needed.
This was not done in the current implementation and thus this was causing
a memory leak.

This commit fixes this by explicitly `free`ing the resources after usage.

Fixes #4910

Co-Authored-By: Zero0one1 <summerlinasity@gmail.com>
Co-Authored-By: Krzmbrzl <krzmbrzl@gmail.com>
2021-04-06 16:45:58 +08:00
Robert Adam
9b070db5da
Merge PR #4917: CI(azure): Clean entire workspace before starting
On our self-hosted macOS runner we were seeing issues due to the
workspace not being cleaned up after each run, causing the next one to
fail.

This commit makes sure that Azure will clean the entire workspace of any
remaining files, before starting a new job.

Ref: docs.microsoft.com/en-us/azure/devops/pipelines/process/phases?view=azure-devops&tabs=yaml#workspace
2021-04-06 09:59:26 +02:00
Robert Adam
44d8d92fab FIX(server): Fix undefined behvior on Linux hosts
The fields of the rlimit struct were assumed to be of type long int when
printing via the qWarning function. This assumption is correct for glibc
(used by e.g. Ubuntu) but it is not true for musl (used by e.g. Alpine
Linux).

In that case the wrong format specifiers are being used for printing
these values, which causes undefined behavior.

The fix is to always copy the values into variables of type long long
unsigned int and then printing these. The code also adds an assertion
making sure that within this copy operation the values won't overflow
the variable's size.

Fixes #4897
2021-04-06 09:43:57 +02:00
Robert Adam
c5db8276b3 CI(azure): Clean entire workspace before starting
On our self-hosted macOS runner we were seeing issues due to the
workspace not being cleaned up after each run, causing the next one to
fail.

This commit makes sure that Azure will clean the entire workspace of any
remaining files, before starting a new job.

Ref: https://docs.microsoft.com/en-us/azure/devops/pipelines/process/phases?view=azure-devops&tabs=yaml#workspace
2021-04-06 09:11:20 +02:00
Robert Adam
5ca456ca8e
Merge PR #4914: CHANGE(client, macOS): Change echo cancellation strategy
The current implementation for echo cancellation using Apple's acoustic
echo cancellation system has been reported to cause major problems
(users no longer being able to transmit any audio) and therefore we
disable it by default.

Furthermore we mark the option as experimental so that (hopefully) only
experienced users will use it.

Ref: #4912
2021-04-06 07:51:24 +02:00
Robert Adam
5509aa2f79 TRANSLATION: Update translation files 2021-04-05 20:34:47 +02:00
Robert Adam
bec9085b3e CHANGE(client, ui): Mark Apple echo cancellation as experimental
The implementation as it stands can cause major issues (users not being
able to transmit any audio).

Ref: #4912
2021-04-05 20:34:47 +02:00
Robert Adam
5324425764 CHANGE(client, audio, macOS): Disable echo cancellation by default
The current implementation for echo cancellation using Apple's acoustic
echo cancellation system has been reported to cause major problems
(users no longer being able to transmit any audio) and therefore we
disable it by default.

Ref: #4912
2021-04-05 20:34:47 +02:00
Robert Adam
7e64683554
Merge pull request #4913: MAINT: Add CHANGE commit type
This commit type is supposed to be used when e.g. a setting is changed.
2021-04-05 20:34:24 +02:00
Robert Adam
eabee58e68 MAINT: Add CHANGE commit type
This commit type is supposed to be used when e.g. a setting is changed.
2021-04-05 18:06:48 +02:00
Robert Adam
fbb2a8c823
Merge PR #4903: FIX: Replace RAND_bytes with CryptographicRandom::fillBuffer
openssl.org/docs/man1.1.1/man3/RAND_bytes.html:
If the entropy source fails or is not available, the CSPRNG will enter an error state and refuse to generate random bytes. For that reason, it is important to always check the error return value of RAND_bytes() and RAND_priv_bytes() and not take randomness for granted.
2021-04-04 15:07:17 +02:00
Robert Adam
6ee5afcd88
Merge PR #4904: MAINT: Fix endless loop in translation update script 2021-04-04 12:53:24 +02:00
Robert Adam
e1943b9982 MAINT: Fix endless loop in translation update script 2021-04-03 21:30:33 +02:00
TredwellGit
c454f87072 FIX: Replace RAND_bytes with CryptographicRandom::fillBuffer
https://www.openssl.org/docs/man1.1.1/man3/RAND_bytes.html:
If the entropy source fails or is not available, the CSPRNG will enter an error state and refuse to generate random bytes. For that reason, it is important to always check the error return value of RAND_bytes() and RAND_priv_bytes() and not take randomness for granted.

Also remove unnecessary cast.
2021-04-03 18:24:28 +00:00
Robert Adam
e0d005699e
Merge pull request #4902: FIX(client, markdown): Wrong line break HTML code
The Markdown code was using </br> instead of the correct <br/>.

This error was introduced in 2da3f0d.

Fixes #4899
2021-04-03 14:24:11 +02:00
Robert Adam
c81d1903e1
Merge PR #4901: FIX(positional-audio): Update Among Us plugin to work with v2021.3.31.3s 2021-04-03 09:22:50 +02:00
Robert Adam
0a95dea3e1 FIX(client, markdown): Wrong line break HTML code
The Markdown code was using </br> instead of the correct <br/>.

This error was introduced in 2da3f0d39e.

Fixes #4899
2021-04-03 09:18:58 +02:00
Davide Beatrici
6ceeef39ae FIX(positional-audio): Update Among Us plugin to work with v2021.3.31.3s
https://store.steampowered.com/news/app/945360/view/3021326294554201922
2021-04-03 07:26:02 +02:00
Robert Adam
06f3b32278
Merge pull request #4892: MAINT/CI: Various improvements 2021-03-30 15:17:07 +02:00
Jan Klass
13b8019ead
Merge PR #4878: Translations update from Weblate 2021-03-30 11:49:22 +02:00
Jan Klass
5a2518f081 TRANSLATION: Update translation files 2021-03-30 08:55:24 +02:00
Hosted Weblate
6a389181f6 TRANSLATION: Update Spanish translation
Currently translated at 96.7% (1894 of 1958 strings)

Co-authored-by: Juan Iribarren <iribarrenjg@gmail.com>
Translate-URL: https://hosted.weblate.org/projects/mumble/mumble-client/es/
Translation: Mumble/Mumble Client
2021-03-30 08:53:36 +02:00
Hosted Weblate
c295e42fde TRANSLATION: Update Italian translation
Currently translated at 100.0% (1958 of 1958 strings)

Co-authored-by: Alessandro Mandelli <mandelli.alessandro@ngi.it>
Co-authored-by: Hosted Weblate <hosted@weblate.org>
Translate-URL: https://hosted.weblate.org/projects/mumble/mumble-client/it/
Translation: Mumble/Mumble Client
2021-03-30 08:53:35 +02:00
Hosted Weblate
4b7d9fec19 TRANSLATION: Update Polish translation
Currently translated at 100.0% (1958 of 1958 strings)

Co-authored-by: Hosted Weblate <hosted@weblate.org>
Co-authored-by: Matthaiks <kitynska@gmail.com>
Translate-URL: https://hosted.weblate.org/projects/mumble/mumble-client/pl/
Translation: Mumble/Mumble Client
2021-03-30 08:53:34 +02:00
Hosted Weblate
21cb8a3289 TRANSLATION: Update Portuguese (Brazil) translation
Currently translated at 99.3% (1945 of 1958 strings)

Co-authored-by: Hosted Weblate <hosted@weblate.org>
Co-authored-by: Vitor Sabino Cardoso da SIlva <vitorcsilva180396@gmail.com>
Translate-URL: https://hosted.weblate.org/projects/mumble/mumble-client/pt_BR/
Translation: Mumble/Mumble Client
2021-03-30 08:53:32 +02:00
Hosted Weblate
e53f828721 TRANSLATION: Update Romanian translation
Currently translated at 9.0% (177 of 1958 strings)

Co-authored-by: Alex Gellen <alex.n.gellen@gmail.com>
Translate-URL: https://hosted.weblate.org/projects/mumble/mumble-client/ro/
Translation: Mumble/Mumble Client
2021-03-30 08:53:30 +02:00
Hosted Weblate
38f69e92a5 TRANSLATION: Update Russian translation
Currently translated at 99.7% (1954 of 1958 strings)

Co-authored-by: Hosted Weblate <hosted@weblate.org>
Co-authored-by: Nikita Epifanov <nikgreens@protonmail.com>
Translate-URL: https://hosted.weblate.org/projects/mumble/mumble-client/ru/
Translation: Mumble/Mumble Client
2021-03-30 08:53:28 +02:00
Hosted Weblate
a265661e82 TRANSLATION: Update Swedish translation
Currently translated at 91.8% (1798 of 1958 strings)

Co-authored-by: Hosted Weblate <hosted@weblate.org>
Co-authored-by: Kristoffer Grundström <swedishsailfishosuser@tutanota.com>
Co-authored-by: Shjosan <shjosan@kakmix.co>
Translate-URL: https://hosted.weblate.org/projects/mumble/mumble-client/sv/
Translation: Mumble/Mumble Client
2021-03-30 08:53:26 +02:00
Hosted Weblate
6caf01de59 TRANSLATION: Update Turkish translation
Currently translated at 90.3% (1770 of 1958 strings)

Co-authored-by: Hosted Weblate <hosted@weblate.org>
Co-authored-by: Oğuz Ersen <oguzersen@protonmail.com>
Translate-URL: https://hosted.weblate.org/projects/mumble/mumble-client/tr/
Translation: Mumble/Mumble Client
2021-03-30 08:53:20 +02:00
Hosted Weblate
7f938bf588 TRANSLATION: Update Chinese (Simplified) translation
Currently translated at 100.0% (1958 of 1958 strings)

Co-authored-by: Hosted Weblate <hosted@weblate.org>
Co-authored-by: deluxghost <deluxghost@gmail.com>
Translate-URL: https://hosted.weblate.org/projects/mumble/mumble-client/zh_Hans/
Translation: Mumble/Mumble Client
2021-03-30 08:53:14 +02:00
Hosted Weblate
e4131ce34c TRANSLATION: Update German translation
Currently translated at 99.4% (1948 of 1958 strings)

Co-authored-by: Hosted Weblate <hosted@weblate.org>
Co-authored-by: Jörg S <joerg.schwerdtfeger@gmail.com>
Co-authored-by: Translator <weblate.gledy@aleeas.com>
Translate-URL: https://hosted.weblate.org/projects/mumble/mumble-client/de/
Translation: Mumble/Mumble Client
2021-03-30 08:53:11 +02:00
Hosted Weblate
d7a70d596b TRANSLATION: Update Dutch translation
Currently translated at 100.0% (1958 of 1958 strings)

Co-authored-by: Gert-dev <qnyasgjhapqyuhoibr@kiabws.com>
Co-authored-by: Hosted Weblate <hosted@weblate.org>
Translate-URL: https://hosted.weblate.org/projects/mumble/mumble-client/nl/
Translation: Mumble/Mumble Client
2021-03-30 08:52:45 +02:00
Robert Adam
c3aa37237d
Merge pull request #4891: FEAT(client, ui): Revamped server information dialog
The previous dialog was simply a message box with a bunch of HTML in
order to give a little structure to the contents. This approach is very
limiting in terms of UI flexibility though and therefore this commit
replaces the old HTML approach with a dedicated dialog class that uses
proper UI elements instead.

While doing so, the ordering and grouping of information was also
changed in order to make it more suitable for the every-day-user.

Fixes #4733
2021-03-28 21:58:23 +02:00
Robert Adam
c0a71a7f3d TRANSLATION: Update translation files 2021-03-28 20:30:06 +02:00
Robert Adam
5fabceee9d FEAT(client, ui): Revamped server information dialog
The previous dialog was simply a message box with a bunch of HTML in
order to give a little structure to the contents. This approach is very
limiting in terms of UI flexibility though and therefore this commit
replaces the old HTML approach with a dedicated dialog class that uses
proper UI elements instead.

While doing so, the ordering and grouping of information was also
changed in order to make it more suitable for the every-day-user.

Fixes #4733
2021-03-28 20:23:38 +02:00
Robert Adam
b48ca7caec CI: Added check to prevent CRLF endings 2021-03-28 20:17:38 +02:00
Robert Adam
2902913d68 MAINT: Loop until there are no more translation changes
Before it could happen that one had to run the script twice in a row in
order to make sure e.g. sametext heuristics are applied.
2021-03-28 20:17:38 +02:00
Robert Adam
0c26ff14b5 FORMAT: Removed CRLF line endings 2021-03-28 20:17:24 +02:00
Robert Adam
0343d2b002 MAINT/CI: Add --ci-mode option to translation update script
In this mode the sametext heuristic is disabled which _should_ fix the
issue of the Weblate PR's CI failing because of Weblate not applying
such heuristics.
2021-03-28 19:58:54 +02:00