Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Make room_metadata Prosody module depend on the required jitsi_session module #14012

Merged
merged 1 commit into from
Nov 1, 2023

Conversation

ntninja
Copy link
Contributor

@ntninja ntninja commented Nov 1, 2023

Without this room_metadata will silently discard all room metadata client requests assuming that they didn’t come from Jitsi meet clients.

Fixes #14001

Detailed description in that issue.

…ession module

Without this room_metadata will silently discard all room metadata client requests assuming that they didn’t come from Jitsi meet clients.

Fixes jitsi#14001
@jitsi-jenkins
Copy link

Hi, thanks for your contribution!
If you haven't already done so, could you please make sure you sign our CLA (https://jitsi.org/icla for individuals and https://jitsi.org/ccla for corporations)? We would unfortunately be unable to merge your patch unless we have that piece :(.

@ntninja
Copy link
Contributor Author

ntninja commented Nov 1, 2023

CLA submitted (name Yuki Schlarb) – never had a system so blatantly demo how pointless “handwritten” signatures are in electronic communication 😮 😁

(If you don’t know what I mean, the CLA submission system actually offers to generate a convincing looking handwritten signature image based on the name you entered for you – it’s even the default option and get to pick a writing style too!)

@damencho
Copy link
Member

damencho commented Nov 1, 2023

jenkins test this please.

@damencho damencho added the info:cla-signed The CLA has been signed label Nov 1, 2023
@damencho damencho merged commit 2851eee into jitsi:master Nov 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
info:cla-signed The CLA has been signed
Projects
None yet
4 participants