Skip to content
This repository was archived by the owner on Jan 30, 2025. It is now read-only.

Read jitsi server url from .well-known configuration#3456

Open
Targunitoth wants to merge 3 commits into
element-hq:developfrom
Targunitoth:develop
Open

Read jitsi server url from .well-known configuration#3456
Targunitoth wants to merge 3 commits into
element-hq:developfrom
Targunitoth:develop

Conversation

@Targunitoth

Copy link
Copy Markdown

Pull Request Checklist

PR for Issue #3443, depends on SDK Update
It would be kind if you could test and improve my PR.

  • Changes has been tested on an Android device or Android emulator with API 16
  • UI change has been tested on both light and dark themes
  • Pull request is based on the develop branch
  • Pull request updates CHANGES.rst
  • Pull request includes screenshots or videos if containing UI changes
  • Pull request includes a sign off

@Targunitoth

Targunitoth commented Apr 30, 2020

Copy link
Copy Markdown
Author

I don't know, what I did wrong to get the Travis Cl build fail.
Also I testet my implementation and it seems like there is still a bug, the initUiAndData() Function is not always called.
The Problem is to get the non-static String mSession.getHomeServerConfig().getJitsiServerUri().toString(); to the static variable JITSI_SERVER_URL, which is used in 2 companion kotlin functions.

@bmarty

bmarty commented May 13, 2020

Copy link
Copy Markdown
Member

Unfortunately, the Wellknown configuration returned when user logs in does not contains all the fields that are provided in a classical Wellknown request.
The doc has been updated: https://matrix.org/docs/spec/client_server/latest#post-matrix-client-r0-login

So I think this will not work. Was it working on your side?

@Targunitoth

Copy link
Copy Markdown
Author

Maybe an SDK Upgrade was needed. Also I think this PR is obsolete due to #3462.

@MisterRager

Copy link
Copy Markdown
Contributor

We definitely need this patch, too. The one I added is only good for finding currently active widgets, not for gathering information on how to widget if it's not there yet. Sorry for the slow response - I didn't have my personal laptop for a bit.

I could fix the patch if you want to get it landed faster.

@Targunitoth

Copy link
Copy Markdown
Author

Feel free to fix it. I'm currently short on time.
The missing link is something like:
JITSI_SERVER_URL = mSession.getHomeServerConfig().getJitsiServerUri().toString();
To get a valid session and then get the non-static value into the static variable.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants