Show organization is an extra privacy feature

FAQ.md

@@ -1458,6 +1458,7 @@ FairEmail will try to select the best identity based on the *to* address of the
 
 The advanced option *extra privacy features* enables:
 
+* Looking up the owner of the IP address of a link
 * Detection and removal of [tracking images](#user-content-faq82)
 * Removal of [Urchin Tracking Module (UTM) parameters](https://en.wikipedia.org/wiki/UTM_parameters) from links
 

app/src/main/java/eu/faircode/email/AdapterMessage.java

@@ -1847,48 +1847,43 @@ public class AdapterMessage extends RecyclerView.Adapter<AdapterMessage.ViewHold
 
                 View view = LayoutInflater.from(context).inflate(R.layout.dialog_link, null);
                 final EditText etLink = view.findViewById(R.id.etLink);
-                final CheckBox cbOrganization = view.findViewById(R.id.cbOrganization);
                 TextView tvInsecure = view.findViewById(R.id.tvInsecure);
+                final TextView tvOwner = view.findViewById(R.id.tvOwner);
+                Group grpOwner = view.findViewById(R.id.grpOwner);
 
-                cbOrganization.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() {
+                etLink.setText(_uri.toString());
-                    @Override
+                tvInsecure.setVisibility("http".equals(_uri.getScheme()) ? View.VISIBLE : View.GONE);
-                    public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {
+                grpOwner.setVisibility(paranoid ? View.VISIBLE : View.GONE);
-                        prefs.edit().putBoolean("show_organization", isChecked).apply();
-                        if (isChecked) {
-                            Bundle args = new Bundle();
-                            args.putParcelable("uri", _uri);
 
-                            new SimpleTask<String>() {
+                if (paranoid) {
-                                @Override
+                    Bundle args = new Bundle();
-                                protected void onPreExecute(Bundle args) {
+                    args.putParcelable("uri", _uri);
-                                    cbOrganization.setText("…");
-                                }
 
-                                @Override
+                    new SimpleTask<String>() {
-                                protected String onExecute(Context context, Bundle args) throws Throwable {
+                        @Override
-                                    Uri uri = args.getParcelable("uri");
+                        protected void onPreExecute(Bundle args) {
-                                    String host = uri.getHost();
+                            tvOwner.setText("…");
-                                    return (TextUtils.isEmpty(host) ? null : Helper.getOrganization(host));
+                        }
-                                }
 
-                                @Override
+                        @Override
-                                protected void onExecuted(Bundle args, String organization) {
+                        protected String onExecute(Context context, Bundle args) throws Throwable {
-                                    cbOrganization.setText(organization == null ? "?" : organization);
+                            Uri uri = args.getParcelable("uri");
-                                }
+                            String host = uri.getHost();
+                            return (TextUtils.isEmpty(host) ? null : Helper.getOrganization(host));
+                        }
 
-                                @Override
+                        @Override
-                                protected void onException(Bundle args, Throwable ex) {
+                        protected void onExecuted(Bundle args, String organization) {
-                                    cbOrganization.setText(ex.getMessage());
+                            tvOwner.setText(organization == null ? "?" : organization);
-                                }
+                        }
-                            }.execute(context, owner, args, "link:domain");
+
-                        } else
+                        @Override
-                            cbOrganization.setText(R.string.title_show_organization);
+                        protected void onException(Bundle args, Throwable ex) {
-                    }
+                            tvOwner.setText(ex.getMessage());
-                });
+                        }
+                    }.execute(context, owner, args, "link:domain");
+                }
 
-                etLink.setText(_uri.toString());
-                cbOrganization.setChecked(prefs.getBoolean("show_organization", true));
-                tvInsecure.setVisibility("http".equals(_uri.getScheme()) ? View.VISIBLE : View.GONE);
 
                 new DialogBuilderLifecycle(context, owner)
                         .setView(view)

app/src/main/res/layout/dialog_link.xml

@@ -28,24 +28,42 @@
         app:layout_constraintStart_toStartOf="parent"
         app:layout_constraintTop_toBottomOf="@id/tvOpenLink" />
 
-    <CheckBox
+    <TextView
-        android:id="@+id/cbOrganization"
+        android:id="@+id/tvInsecure"
         android:layout_width="wrap_content"
         android:layout_height="wrap_content"
-        android:layout_marginTop="6dp"
+        android:layout_marginTop="12dp"
-        android:text="@string/title_show_organization"
+        android:text="@string/title_insecure_link"
+        android:textAppearance="@style/TextAppearance.AppCompat.Small"
+        android:textColor="?attr/colorWarning"
+        android:textStyle="bold"
         app:layout_constraintStart_toStartOf="parent"
         app:layout_constraintTop_toBottomOf="@id/etLink" />
 
     <TextView
-        android:id="@+id/tvInsecure"
+        android:id="@+id/tvOwnerTitle"
         android:layout_width="wrap_content"
         android:layout_height="wrap_content"
         android:layout_marginTop="12dp"
-        android:text="@string/title_insecure_link"
+        android:labelFor="@+id/etLink"
+        android:text="@string/title_ip_owner"
         android:textAppearance="@style/TextAppearance.AppCompat.Small"
-        android:textColor="?attr/colorWarning"
-        android:textStyle="bold"
         app:layout_constraintStart_toStartOf="parent"
-        app:layout_constraintTop_toBottomOf="@id/cbOrganization" />
+        app:layout_constraintTop_toBottomOf="@id/tvInsecure" />
+
+    <TextView
+        android:id="@+id/tvOwner"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        android:labelFor="@+id/etLink"
+        android:text="Cloudflare"
+        android:textAppearance="@style/TextAppearance.AppCompat.Medium"
+        app:layout_constraintStart_toStartOf="parent"
+        app:layout_constraintTop_toBottomOf="@id/tvOwnerTitle" />
+
+    <androidx.constraintlayout.widget.Group
+        android:id="@+id/grpOwner"
+        android:layout_width="0dp"
+        android:layout_height="0dp"
+        app:constraint_referenced_ids="tvOwnerTitle,tvOwner" />
 </androidx.constraintlayout.widget.ConstraintLayout>

app/src/main/res/values/strings.xml

@@ -593,7 +593,7 @@
     <string name="title_hint_ignore_contact">Long press a contact to never consider it as a favorite</string>
 
     <string name="title_open_link">Open link</string>
-    <string name="title_show_organization">Show organization</string>
+    <string name="title_ip_owner">Owner IP address</string>
     <string name="title_insecure_link">This link is insecure</string>
     <string name="title_select_app">Select app</string>
     <string name="title_updated">There is an update to version %1$s available</string>