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 connectivity issues, switch from http-bind to websocket #155

Merged
merged 6 commits into from
Jan 31, 2024
Merged

Conversation

p-fruck
Copy link
Member

@p-fruck p-fruck commented Jan 31, 2024

Description

Fixes the connectivity issues reported in #154. The bosh parameter has been changed to serviceUrl in lib-jitsi-meet which supports websockets instead of http-bind. Websocket are apparently not subject to CORS so the http-bind related stuff has been removed from the browser extension. The extension is now solely required for streaming from google video, meaning that Jimmi could be used as a Chatbot without the plugin.

Checklist

  • Make sure you are requesting to dev. Don't request to main branch!
  • I have linked related issue.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

dependabot bot and others added 6 commits January 20, 2024 05:02
Bumps [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite) from 4.5.0 to 4.5.2.
- [Release notes](https://github.com/vitejs/vite/releases)
- [Changelog](https://github.com/vitejs/vite/blob/v4.5.2/packages/vite/CHANGELOG.md)
- [Commits](https://github.com/vitejs/vite/commits/v4.5.2/packages/vite)

---
updated-dependencies:
- dependency-name: vite
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <[email protected]>
anonymousdomain is required to login as guest.
The serviceUrl has been changed to websocket which
should be more efficient than http-bind
After changing to websocket instead of http-bind
this rule is not required anymore since websockets
are not effected by CORS. The extension is now
only required for streaming from googlevideo
Switched to websocket from http-bind, see previous
commits
…arn/frontend/vite-4.5.2

Bump vite from 4.5.0 to 4.5.2 in /frontend
@p-fruck p-fruck added the bug Something isn't working label Jan 31, 2024
@p-fruck p-fruck self-assigned this Jan 31, 2024
@p-fruck p-fruck merged commit aa7825d into main Jan 31, 2024
1 check passed
@p-fruck p-fruck deleted the dev branch January 31, 2024 23:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant