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

Missing device_id in matrix-android-sdk2 login API #4334

Closed
hugohn opened this issue Oct 25, 2021 · 7 comments · Fixed by #4344
Closed

Missing device_id in matrix-android-sdk2 login API #4334

hugohn opened this issue Oct 25, 2021 · 7 comments · Fixed by #4344
Assignees
Labels
matrix-sdk T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements

Comments

@hugohn
Copy link

hugohn commented Oct 25, 2021

Description

According to the current spec, the Matrix login API takes in a device_id, but matrix-android-sdk2, which Element Android uses, doesn't seem to expose the device_id param for logging in. There is a PasswordLoginParams class with device_id, but it is an internal class to the SDK. Does anyone know how to log in with device_id?

I checked libQuotient and matrix-ios-sdk and they both provide the login API with a device_id param (explicitly in libQuotient and via a dictionary in matrix-ios-sdk).

@RiotRobot RiotRobot added this to Incoming in Issue triage Oct 25, 2021
@bmarty bmarty added this to Next Sprint in Android App Team Oct 25, 2021
@bmarty bmarty added this to the Sprint - Element 1.3.5 milestone Oct 25, 2021
@bmarty
Copy link
Member

bmarty commented Oct 25, 2021

You are right, providing a deviceId is not currently possible in the Matrix SDK2 API.
There is really not a big deal to change that, let's do that in the next sprint:
Add a parameter deviceId: String? = null to this method

@hugohn
Copy link
Author

hugohn commented Oct 26, 2021

Awesome. Thank you @bmarty !

@hugohn
Copy link
Author

hugohn commented Oct 26, 2021

@bmarty would it be possible to add deviceId to AuthenticationService.directAuthentication() too?

https://github.com/vector-im/element-android/blob/main/matrix-sdk-android/src/main/java/org/matrix/android/sdk/api/auth/AuthenticationService.kt#L109

@bmarty bmarty moved this from Next Sprint to Current Sprint in Android App Team Oct 26, 2021
@bmarty
Copy link
Member

bmarty commented Oct 26, 2021

Yes, will will add it wherever it makes thanks, thanks for the reminder.

@bmarty bmarty self-assigned this Oct 26, 2021
@ouchadam ouchadam added the T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements label Oct 26, 2021
@ouchadam ouchadam moved this from Incoming to Triaged in Issue triage Oct 26, 2021
@bmarty
Copy link
Member

bmarty commented Oct 26, 2021

@hugohn #4344 should cover your needs. Please yell if it's not the case!

@hugohn
Copy link
Author

hugohn commented Oct 26, 2021

@bmarty it looks good to me!

Issue triage automation moved this from Triaged to Closed Oct 26, 2021
Android App Team automation moved this from Current Sprint to Merged Oct 26, 2021
@bmarty
Copy link
Member

bmarty commented Oct 26, 2021

This will be in SDK 1.3.7

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
matrix-sdk T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements
Projects
Development

Successfully merging a pull request may close this issue.

3 participants