Enable Login v2 #2908
Enable Login v2 #2908
Conversation
Signed-off-by: Milen Pivchev <milen.pivchev@gmail.com>
Brand/NCBrand.swift
Outdated
| @@ -68,7 +68,7 @@ let userAgent: String = { | |||
| @objc public var use_themingColor: Bool = true | |||
| @objc public var use_themingLogo: Bool = false | |||
| @objc public var use_storeLocalAutoUploadAll: Bool = false | |||
| @objc public var use_loginflowv2: Bool = false // Don't touch me !! | |||
| @objc public var use_loginflowv2: Bool = true // Don't touch me !! | |||
There was a problem hiding this comment.
@marinofaggiana The plan is to use v2 by default from now on, and v1 will be obsolete with removal of oldest supported NC version. Maybe we can remove this? Not sure if the brander will break
There was a problem hiding this comment.
yes I created this switch years ago to test v2
There was a problem hiding this comment.
Is it ok to remove it?
Signed-off-by: Milen Pivchev <milen.pivchev@gmail.com>
|
Added the option to do the flow via external browser @tobiasKaminsky |
iOSClient/Login/NCLoginWeb.swift
Outdated
| @@ -47,6 +47,9 @@ class NCLoginWeb: UIViewController { | |||
| var loginFlowV2Endpoint = "" | |||
| var loginFlowV2Login = "" | |||
|
|
|||
| // Opens the login URL in external browser instead of in app. User must manually go back to the app. | |||
| let loginFlowv2ExternalBrowser = false | |||
There was a problem hiding this comment.
@marinofaggiana Please test with this enabled and disabled
iOSClient/Login/NCLoginWeb.swift
Outdated
| NextcloudKit.shared.getLoginFlowV2Poll(token: self.loginFlowV2Token, endpoint: self.loginFlowV2Endpoint) { server, loginName, appPassword, _, error in | ||
| if error == .success, let server, let loginName, let appPassword { | ||
| self.createAccount(server: server, username: loginName, password: appPassword) | ||
| } |
There was a problem hiding this comment.
If error write a message and goBack ?
There was a problem hiding this comment.
I stopped using NCLoginWeb for v2 and this screen stopped appearing. Now we only use NCLogin. NCLoginWeb can be renamed to NCLoginProviders as now only external providers use it.
Signed-off-by: Milen Pivchev <milen.pivchev@gmail.com>
|
@marinofaggiana Please check |
Signed-off-by: Milen Pivchev <milen.pivchev@gmail.com> # Conflicts: # iOSClient/Networking/NCNetworking.swift
Signed-off-by: Milen Pivchev <milen.pivchev@gmail.com>
Signed-off-by: Milen Pivchev <milen.pivchev@gmail.com>
Signed-off-by: Milen Pivchev <milen.pivchev@gmail.com>
Signed-off-by: Milen Pivchev <milen.pivchev@gmail.com>
Signed-off-by: Milen Pivchev <milen.pivchev@gmail.com> # Conflicts: # iOSClient/Login/NCLogin.swift # iOSClient/Supporting Files/en.lproj/Localizable.strings
| } | ||
| } | ||
| } | ||
| } | ||
| } | ||
|
|
…-v2-enable Signed-off-by: Milen Pivchev <milen.pivchev@gmail.com> # Conflicts: # Brand/NCBrand.swift # Nextcloud.xcodeproj/project.pbxproj # iOSClient/AppDelegate.swift
Signed-off-by: Milen Pivchev <milen.pivchev@gmail.com>
This enables Login v2 and adds support for TLS client certificates.
@marinofaggiana We can remove all v1 code with removal of oldest supported NC version