Skip to content

feat(transport): add EmbeddedTransportBackend for no-iframe embedding#130

Closed
mishraditi wants to merge 3 commits into
jitsi:masterfrom
mishraditi:embedded-transport-backend
Closed

feat(transport): add EmbeddedTransportBackend for no-iframe embedding#130
mishraditi wants to merge 3 commits into
jitsi:masterfrom
mishraditi:embedded-transport-backend

Conversation

@mishraditi

Copy link
Copy Markdown

Description:
Adds EmbeddedTransportBackend - a new ITransportBackend that communicates through direct function calls instead of postMessage across an iframe.

It sits here alongside PostMessageTransportBackend since this is where the transport layer and ITransportBackend interface already live.

Changes:

  • EmbeddedTransportBackend.ts - new backend, use static createPair() to get a linked pair of instances.
  • EmbeddedTransportBackend.spec.ts - 12 unit tests covering pairing, bidirectional messaging,
    dispose, and Transport integration.
  • transport/index.ts - exports EmbeddedTransportBackend alongside the existing backends.

This is part of the "External API without iframe" project. Follow-up PRs in jitsi-meet will use this to render Jitsi Meet directly into a host page div without an iframe.

@mishraditi

Copy link
Copy Markdown
Author

@saghul @hristoterezov,
Closing this, after the discussion on jitsi/jitsi-meet#17477 the embedded API doesn't use a transport anymore. Both sides run in the same page and call each other directly, so this loopback backend isn't needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant