From 6cdb192955ed1933629eb39895e81108ce654d1e Mon Sep 17 00:00:00 2001 From: Benoit Marty Date: Tue, 15 Dec 2020 14:18:35 +0100 Subject: [PATCH 1/7] Version++ --- CHANGES.md | 27 +++++++++++++++++++++++++++ vector/build.gradle | 2 +- 2 files changed, 28 insertions(+), 1 deletion(-) diff --git a/CHANGES.md b/CHANGES.md index 83c825c21e..a2029b9901 100644 --- a/CHANGES.md +++ b/CHANGES.md @@ -1,3 +1,30 @@ +Changes in Element 1.0.13 (2020-XX-XX) +=================================================== + +Features ✨: + - + +Improvements 🙌: + - + +Bugfix 🐛: + - + +Translations 🗣: + - + +SDK API changes ⚠️: + - + +Build 🧱: + - + +Test: + - + +Other changes: + - + Changes in Element 1.0.12 (2020-12-15) =================================================== diff --git a/vector/build.gradle b/vector/build.gradle index 7bb8ca187c..70d38b1c2a 100644 --- a/vector/build.gradle +++ b/vector/build.gradle @@ -17,7 +17,7 @@ androidExtensions { // Note: 2 digits max for each value ext.versionMajor = 1 ext.versionMinor = 0 -ext.versionPatch = 12 +ext.versionPatch = 13 static def getGitTimestamp() { def cmd = 'git show -s --format=%ct' From ea6fde3ed0fd0a04abc36fa94196615b209533ec Mon Sep 17 00:00:00 2001 From: Travis Ralston Date: Tue, 15 Dec 2020 11:42:53 -0700 Subject: [PATCH 2/7] Make MSC2858 implementation match the unstable policy of the spec See https://github.com/matrix-org/matrix-doc/pull/2858/files#r543567196 --- .../main/java/org/matrix/android/sdk/api/auth/Constants.kt | 1 + .../android/sdk/internal/auth/data/LoginFlowResponse.kt | 2 +- .../main/java/im/vector/app/features/login/LoginViewState.kt | 4 +++- 3 files changed, 5 insertions(+), 2 deletions(-) diff --git a/matrix-sdk-android/src/main/java/org/matrix/android/sdk/api/auth/Constants.kt b/matrix-sdk-android/src/main/java/org/matrix/android/sdk/api/auth/Constants.kt index 871c2559f8..7d18aba627 100644 --- a/matrix-sdk-android/src/main/java/org/matrix/android/sdk/api/auth/Constants.kt +++ b/matrix-sdk-android/src/main/java/org/matrix/android/sdk/api/auth/Constants.kt @@ -33,5 +33,6 @@ const val REGISTER_FALLBACK_PATH = "/_matrix/static/client/register/" * Ref: https://matrix.org/docs/spec/client_server/latest#sso-client-login */ const val SSO_REDIRECT_PATH = "/_matrix/client/r0/login/sso/redirect" +const val MSC2858_SSO_REDIRECT_PATH = "/_matrix/client/unstable/org.matrix.msc2858/login/sso/redirect" const val SSO_REDIRECT_URL_PARAM = "redirectUrl" diff --git a/matrix-sdk-android/src/main/java/org/matrix/android/sdk/internal/auth/data/LoginFlowResponse.kt b/matrix-sdk-android/src/main/java/org/matrix/android/sdk/internal/auth/data/LoginFlowResponse.kt index c333b3524e..2b26115f30 100644 --- a/matrix-sdk-android/src/main/java/org/matrix/android/sdk/internal/auth/data/LoginFlowResponse.kt +++ b/matrix-sdk-android/src/main/java/org/matrix/android/sdk/internal/auth/data/LoginFlowResponse.kt @@ -42,6 +42,6 @@ internal data class LoginFlow( * the client can show a button for each of the supported providers * See MSC #2858 */ - @Json(name = "identity_providers") + @Json(name = "org.matrix.msc2858.identity_providers") val ssoIdentityProvider: List? ) diff --git a/vector/src/main/java/im/vector/app/features/login/LoginViewState.kt b/vector/src/main/java/im/vector/app/features/login/LoginViewState.kt index 383fd4a54e..606bd7dc21 100644 --- a/vector/src/main/java/im/vector/app/features/login/LoginViewState.kt +++ b/vector/src/main/java/im/vector/app/features/login/LoginViewState.kt @@ -72,9 +72,11 @@ data class LoginViewState( fun getSsoUrl(providerId: String?): String { return buildString { append(homeServerUrl?.trim { it == '/' }) - append(SSO_REDIRECT_PATH) if (providerId != null) { + append(MSC2858_SSO_REDIRECT_PATH) append("/$providerId") + } else { + append(SSO_REDIRECT_PATH) } // Set a redirect url we will intercept later appendParamToUrl(SSO_REDIRECT_URL_PARAM, VECTOR_REDIRECT_URL) From 3491774e7b9da2b22269b5738062587b37d28a20 Mon Sep 17 00:00:00 2001 From: Travis Ralston Date: Tue, 15 Dec 2020 11:45:06 -0700 Subject: [PATCH 3/7] Add changelog --- CHANGES.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/CHANGES.md b/CHANGES.md index a2029b9901..f1e1061c49 100644 --- a/CHANGES.md +++ b/CHANGES.md @@ -8,7 +8,7 @@ Improvements 🙌: - Bugfix 🐛: - - + - Fix MSC2858 implementation details (#2540) Translations 🗣: - From c7afcf4ff2504589c01ccce796942eb98e43e73c Mon Sep 17 00:00:00 2001 From: Benoit Marty Date: Thu, 17 Dec 2020 12:10:55 +0100 Subject: [PATCH 4/7] Fix compilation issue --- .../src/main/java/im/vector/app/features/login/LoginViewState.kt | 1 + 1 file changed, 1 insertion(+) diff --git a/vector/src/main/java/im/vector/app/features/login/LoginViewState.kt b/vector/src/main/java/im/vector/app/features/login/LoginViewState.kt index 606bd7dc21..e444844022 100644 --- a/vector/src/main/java/im/vector/app/features/login/LoginViewState.kt +++ b/vector/src/main/java/im/vector/app/features/login/LoginViewState.kt @@ -23,6 +23,7 @@ import com.airbnb.mvrx.PersistState import com.airbnb.mvrx.Success import com.airbnb.mvrx.Uninitialized import im.vector.app.core.extensions.appendParamToUrl +import org.matrix.android.sdk.api.auth.MSC2858_SSO_REDIRECT_PATH import org.matrix.android.sdk.api.auth.SSO_REDIRECT_PATH import org.matrix.android.sdk.api.auth.SSO_REDIRECT_URL_PARAM From 68cd06f1fb84217d04efcd4885456aedeeda4c9b Mon Sep 17 00:00:00 2001 From: Benoit Marty Date: Thu, 17 Dec 2020 11:15:19 +0100 Subject: [PATCH 5/7] Improve redirect url, which can be visible to the user: "element://element" -> "element://connect" --- docs/signin.md | 4 ++-- vector/src/main/AndroidManifest.xml | 5 +++-- .../main/java/im/vector/app/features/login/LoginViewState.kt | 2 +- 3 files changed, 6 insertions(+), 5 deletions(-) diff --git a/docs/signin.md b/docs/signin.md index 06f715c46b..0a234d2a20 100644 --- a/docs/signin.md +++ b/docs/signin.md @@ -165,7 +165,7 @@ In this case, the user can click on "Sign in with SSO" and the native web browse > https://homeserver.with.sso/_matrix/client/r0/login/sso/redirect?redirectUrl=element%3A%2F%element -The parameter `redirectUrl` is set to `element://element`. +The parameter `redirectUrl` is set to `element://connect`. ChromeCustomTabs are an intermediate way to display a WebPage, between a WebView and using the external browser. More info can be found [here](https://developer.chrome.com/multidevice/android/customtabs) @@ -175,7 +175,7 @@ During the process, user may be asked to validate an email by clicking on a link Once the process is finished, the web page will call the `redirectUrl` with an extra parameter `loginToken` -> element://element?loginToken=MDAxOWxvY2F0aW9uIG1vemlsbGEub3JnCjAwMTNpZGVudGlmaWVy +> element://connect?loginToken=MDAxOWxvY2F0aW9uIG1vemlsbGEub3JnCjAwMTNpZGVudGlmaWVy This navigation is intercepted by Element by the `LoginActivity`, which will then ask the homeserver to convert this `loginToken` to an access token diff --git a/vector/src/main/AndroidManifest.xml b/vector/src/main/AndroidManifest.xml index bf839b807c..4ad51fdea7 100644 --- a/vector/src/main/AndroidManifest.xml +++ b/vector/src/main/AndroidManifest.xml @@ -95,8 +95,9 @@ - - + diff --git a/vector/src/main/java/im/vector/app/features/login/LoginViewState.kt b/vector/src/main/java/im/vector/app/features/login/LoginViewState.kt index e444844022..5254abf1d9 100644 --- a/vector/src/main/java/im/vector/app/features/login/LoginViewState.kt +++ b/vector/src/main/java/im/vector/app/features/login/LoginViewState.kt @@ -90,6 +90,6 @@ data class LoginViewState( companion object { // Note that the domain can be displayed to the user for confirmation that he trusts it. So use a human readable string - private const val VECTOR_REDIRECT_URL = "element://element" + private const val VECTOR_REDIRECT_URL = "element://connect" } } From 930b8da3b3197c5aea78d66a1c82786ea4340d4b Mon Sep 17 00:00:00 2001 From: Benoit Marty Date: Fri, 18 Dec 2020 09:53:59 +0100 Subject: [PATCH 6/7] Typo in comment --- .../sdk/internal/session/identity/IdentityBulkLookupTask.kt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/matrix-sdk-android/src/main/java/org/matrix/android/sdk/internal/session/identity/IdentityBulkLookupTask.kt b/matrix-sdk-android/src/main/java/org/matrix/android/sdk/internal/session/identity/IdentityBulkLookupTask.kt index 3b0d514cf3..a03bef9501 100644 --- a/matrix-sdk-android/src/main/java/org/matrix/android/sdk/internal/session/identity/IdentityBulkLookupTask.kt +++ b/matrix-sdk-android/src/main/java/org/matrix/android/sdk/internal/session/identity/IdentityBulkLookupTask.kt @@ -93,7 +93,7 @@ internal class DefaultIdentityBulkLookupTask @Inject constructor( } catch (failure: Throwable) { // Catch invalid hash pepper and retry if (canRetry && failure is Failure.ServerError && failure.error.code == MatrixError.M_INVALID_PEPPER) { - // This is not documented, by the error can contain the new pepper! + // This is not documented, but the error can contain the new pepper! if (!failure.error.newLookupPepper.isNullOrEmpty()) { // Store it and use it right now hashDetailResponse.copy(pepper = failure.error.newLookupPepper) From d53650c8ae801c01730a1628f21aff977806fd28 Mon Sep 17 00:00:00 2001 From: Benoit Marty Date: Fri, 18 Dec 2020 10:59:43 +0100 Subject: [PATCH 7/7] Prepare release 1.0.13 --- CHANGES.md | 23 +------------------ .../android/en-US/changelogs/40100130.txt | 2 ++ 2 files changed, 3 insertions(+), 22 deletions(-) create mode 100644 fastlane/metadata/android/en-US/changelogs/40100130.txt diff --git a/CHANGES.md b/CHANGES.md index f1e1061c49..8a7b82c77b 100644 --- a/CHANGES.md +++ b/CHANGES.md @@ -1,30 +1,9 @@ -Changes in Element 1.0.13 (2020-XX-XX) +Changes in Element 1.0.13 (2020-12-18) =================================================== -Features ✨: - - - -Improvements 🙌: - - - Bugfix 🐛: - Fix MSC2858 implementation details (#2540) -Translations 🗣: - - - -SDK API changes ⚠️: - - - -Build 🧱: - - - -Test: - - - -Other changes: - - - Changes in Element 1.0.12 (2020-12-15) =================================================== diff --git a/fastlane/metadata/android/en-US/changelogs/40100130.txt b/fastlane/metadata/android/en-US/changelogs/40100130.txt new file mode 100644 index 0000000000..39715c2910 --- /dev/null +++ b/fastlane/metadata/android/en-US/changelogs/40100130.txt @@ -0,0 +1,2 @@ +Main changes in this version: URL Preview, new Emoji keyboard, new room settings capabilities, and snow for Christmas! +Full changelog: https://github.com/vector-im/element-android/releases/tag/v1.0.12 \ No newline at end of file