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

jitsi-meet mobile apps don't support server side authentication yet #1506

Closed
petervnv opened this issue Apr 17, 2017 · 6 comments
Closed

jitsi-meet mobile apps don't support server side authentication yet #1506

petervnv opened this issue Apr 17, 2017 · 6 comments
Labels
mobile Issue related to any mobile system running Jitsi Meet

Comments

@petervnv
Copy link

Just as the title suggests, if one enables authentication in the server, then mobile clients can not connect, even as guests (have only tested in iOS but suppose the same with Android)

The error messages one gets in iOS app:

2017-02-14 10:15:35.421306 jitsi-meet-react[2275:652424] '[modules/xmpp/strophe.util.js]', ': ', 'Strophe', 0, 'request id 2.1 got 200'
2017-02-14 10:15:35.422 [info][tid:com.facebook.react.JavaScript] '[modules/xmpp/strophe.util.js]', ': ', 'Strophe', 1, '_dataRecv called'
2017-02-14 10:15:35.421852 jitsi-meet-react[2275:652424] '[modules/xmpp/strophe.util.js]', ': ', 'Strophe', 1, '_dataRecv called'
2017-02-14 10:15:35.423 [info][tid:com.facebook.react.JavaScript] '[modules/xmpp/strophe.util.js]', ': ', 'Strophe', 1, '_doDisconnect was called'
2017-02-14 10:15:35.422647 jitsi-meet-react[2275:652424] '[modules/xmpp/strophe.util.js]', ': ', 'Strophe', 1, '_doDisconnect was called'
2017-02-14 10:15:35.423 [info][tid:com.facebook.react.JavaScript] '[modules/xmpp/xmpp.js]', ': ', '(TIME) Strophe disconnected:\t', 0
2017-02-14 10:15:35.422987 jitsi-meet-react[2275:652424] '[modules/xmpp/xmpp.js]', ': ', '(TIME) Strophe disconnected:\t', 0
2017-02-14 10:15:35.424 [error][tid:com.facebook.react.JavaScript] 'CONNECTION FAILED:', 'connection.passwordRequired'
2017-02-14 10:15:35.423676 jitsi-meet-react[2275:652424] 'CONNECTION FAILED:', 'connection.passwordRequired'

@petervnv
Copy link
Author

petervnv commented May 6, 2017

Just wondering if this is something that's being worked on?

@saghul
Copy link
Member

saghul commented May 6, 2017

Yeas and no :-) Feature parity is our goal, but we can't really say when we'll be working on that particular feature.

@petervnv
Copy link
Author

Understood, thanks.

If I have some free time soon I may take a deeper look and start working on it.
Any clues or hints where I should start would be appreciated!

Cheers

@petervnv
Copy link
Author

Anyone care to point me in the right direction while I still have some free time to look at this in detail?

cheers

@lyubomir
Copy link
Contributor

We want to share as much source code as possible between Web and mobile. So tacking this feature like any other before it that has already been ported to mobile would follow the steps bellow:

  1. Identify the specifics of the feature. In this case, what specific kind of server-side authentication are you talking about?

  2. Search the specific keywords in the Web source code. That would most likely be in the modules/ directory of jitsi/jitsi-meet (and/or jitsi/lib-jitsi-meet).

  3. Port the feature. jitsi/jitsi-meet is ported by rewriting in react and redux, jitsi/lib-jitis-meet is ported by minimal shimming, ifing, etc.

@lyubomir lyubomir added the mobile Issue related to any mobile system running Jitsi Meet label Sep 14, 2017
@lyubomir
Copy link
Contributor

As of e08d240 which is shipped in Jitsi Meet for Android v1.10 (and, hopefully, will be shipped in Jitsi Meet for iOS v1.10), the app supports authenticated deployments (aka secure domains).

On a slightly related note to third-party deployments, please beware that self-signed certificates are not supported yet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mobile Issue related to any mobile system running Jitsi Meet
Projects
None yet
Development

No branches or pull requests

3 participants