Merge pull request #3409 from nextcloud/bugfix/3408/fixLoginFailWithUmlautInUsername

Bugfix/3408/fix login fail with umlaut in username

app/src/main/java/com/nextcloud/talk/dagger/modules/RestModule.java

@@ -40,6 +40,7 @@ import java.io.IOException;
 import java.net.CookieManager;
 import java.net.InetSocketAddress;
 import java.net.Proxy;
+import java.nio.charset.StandardCharsets;
 import java.security.KeyStore;
 import java.security.KeyStoreException;
 import java.security.NoSuchAlgorithmException;
@@ -91,7 +92,7 @@ public class RestModule {
     @Provides
     Proxy provideProxy(AppPreferences appPreferences) {
         if (!TextUtils.isEmpty(appPreferences.getProxyType()) && !"No proxy".equals(appPreferences.getProxyType())
-                && !TextUtils.isEmpty(appPreferences.getProxyHost())) {
+            && !TextUtils.isEmpty(appPreferences.getProxyHost())) {
             GetProxyRunnable getProxyRunnable = new GetProxyRunnable(appPreferences);
             Thread getProxyThread = new Thread(getProxyRunnable);
             getProxyThread.start();
@@ -111,10 +112,10 @@ public class RestModule {
     @Provides
     Retrofit provideRetrofit(OkHttpClient httpClient) {
         Retrofit.Builder retrofitBuilder = new Retrofit.Builder()
-                .client(httpClient)
-                .baseUrl("https://nextcloud.com")
-                .addCallAdapterFactory(RxJava2CallAdapterFactory.createWithScheduler(Schedulers.io()))
-                .addConverterFactory(LoganSquareConverterFactory.create());
+            .client(httpClient)
+            .baseUrl("https://nextcloud.com")
+            .addCallAdapterFactory(RxJava2CallAdapterFactory.createWithScheduler(Schedulers.io()))
+            .addConverterFactory(LoganSquareConverterFactory.create());
 
         return retrofitBuilder.build();
     }
@@ -207,11 +208,14 @@ public class RestModule {
             httpClient.proxy(proxy);
 
             if (appPreferences.getProxyCredentials() &&
-                    !TextUtils.isEmpty(appPreferences.getProxyUsername()) &&
-                    !TextUtils.isEmpty(appPreferences.getProxyPassword())) {
-                httpClient.proxyAuthenticator(new HttpAuthenticator(Credentials.basic(
+                !TextUtils.isEmpty(appPreferences.getProxyUsername()) &&
+                !TextUtils.isEmpty(appPreferences.getProxyPassword())) {
+                httpClient.proxyAuthenticator(new HttpAuthenticator(
+                    Credentials.basic(
                         appPreferences.getProxyUsername(),
-                        appPreferences.getProxyPassword()), "Proxy-Authorization"));
+                        appPreferences.getProxyPassword(),
+                        StandardCharsets.UTF_8),
+                    "Proxy-Authorization"));
             }
         }
 
@@ -225,7 +229,7 @@ public class RestModule {
             httpClient.addInterceptor(loggingInterceptor);
         } else if (context.getResources().getBoolean(R.bool.nc_is_debug)) {
             HttpLoggingInterceptor.Logger fileLogger =
-                    s -> LoggingUtils.INSTANCE.writeLogEntryToFile(context, s);
+                s -> LoggingUtils.INSTANCE.writeLogEntryToFile(context, s);
             HttpLoggingInterceptor loggingInterceptor = new HttpLoggingInterceptor(fileLogger);
             loggingInterceptor.setLevel(HttpLoggingInterceptor.Level.BODY);
             loggingInterceptor.redactHeader("Authorization");
@@ -243,11 +247,11 @@ public class RestModule {
         public Response intercept(@NonNull Chain chain) throws IOException {
             Request original = chain.request();
             Request request = original.newBuilder()
-                    .header("User-Agent", ApiUtils.getUserAgent())
-                    .header("Accept", "application/json")
-                    .header("OCS-APIRequest", "true")
-                    .method(original.method(), original.body())
-                    .build();
+                .header("User-Agent", ApiUtils.getUserAgent())
+                .header("Accept", "application/json")
+                .header("OCS-APIRequest", "true")
+                .method(original.method(), original.body())
+                .build();
 
             return chain.proceed(request);
         }
@@ -282,8 +286,8 @@ public class RestModule {
             }
 
             return response.request().newBuilder()
-                    .header(authenticatorType, credentials)
-                    .build();
+                .header(authenticatorType, credentials)
+                .build();
         }
     }
 
@@ -299,12 +303,12 @@ public class RestModule {
         public void run() {
             if (Proxy.Type.valueOf(appPreferences.getProxyType()) == Proxy.Type.SOCKS) {
                 proxy = new Proxy(Proxy.Type.valueOf(appPreferences.getProxyType()),
-                        InetSocketAddress.createUnresolved(appPreferences.getProxyHost(), Integer.parseInt(
-                                appPreferences.getProxyPort())));
+                                  InetSocketAddress.createUnresolved(appPreferences.getProxyHost(), Integer.parseInt(
+                                      appPreferences.getProxyPort())));
             } else {
                 proxy = new Proxy(Proxy.Type.valueOf(appPreferences.getProxyType()),
-                        new InetSocketAddress(appPreferences.getProxyHost(),
-                                Integer.parseInt(appPreferences.getProxyPort())));
+                                  new InetSocketAddress(appPreferences.getProxyHost(),
+                                                        Integer.parseInt(appPreferences.getProxyPort())));
             }
         }
 

app/src/main/java/com/nextcloud/talk/utils/ApiUtils.java

@@ -34,6 +34,7 @@ import com.nextcloud.talk.data.user.model.User;
 import com.nextcloud.talk.models.RetrofitBucket;
 import com.nextcloud.talk.utils.database.user.CapabilitiesUtilNew;
 
+import java.nio.charset.StandardCharsets;
 import java.util.HashMap;
 import java.util.Map;
 
@@ -60,8 +61,8 @@ public class ApiUtils {
     }
 
     /**
-     * @deprecated This is only supported on API v1-3, in API v4+ please use {@link ApiUtils#getUrlForAttendees(int,
-     * String, String)} instead.
+     * @deprecated This is only supported on API v1-3, in API v4+ please use
+     * {@link ApiUtils#getUrlForAttendees(int, String, String)} instead.
      */
     @Deprecated
     public static String getUrlForRemovingParticipantFromConversation(String baseUrl, String roomToken, boolean isGuest) {
@@ -399,7 +400,7 @@ public class ApiUtils {
         if (TextUtils.isEmpty(username) && TextUtils.isEmpty(token)) {
             return null;
         }
-        return Credentials.basic(username, token);
+        return Credentials.basic(username, token, StandardCharsets.UTF_8);
     }
 
     public static String getUrlNextcloudPush(String baseUrl) {