diff --git a/app/src/main/java/org/mozilla/fenix/settings/search/AddSearchEngineFragment.kt b/app/src/main/java/org/mozilla/fenix/settings/search/AddSearchEngineFragment.kt index 3d1f0790c..627b23665 100644 --- a/app/src/main/java/org/mozilla/fenix/settings/search/AddSearchEngineFragment.kt +++ b/app/src/main/java/org/mozilla/fenix/settings/search/AddSearchEngineFragment.kt @@ -171,8 +171,7 @@ class AddSearchEngineFragment : Fragment(), CompoundButton.OnCheckedChangeListen custom_search_engine_search_string_field.error = when { searchString.isEmpty() -> resources.getString(R.string.search_add_custom_engine_error_empty_search_string) - !searchString.contains("%s") - || !searchString.contains("http") -> + !searchString.contains("%s") -> resources.getString(R.string.search_add_custom_engine_error_missing_template) else -> null } diff --git a/app/src/main/java/org/mozilla/fenix/settings/search/SearchStringValidator.kt b/app/src/main/java/org/mozilla/fenix/settings/search/SearchStringValidator.kt index 976170324..3076bef38 100644 --- a/app/src/main/java/org/mozilla/fenix/settings/search/SearchStringValidator.kt +++ b/app/src/main/java/org/mozilla/fenix/settings/search/SearchStringValidator.kt @@ -20,6 +20,8 @@ object SearchStringValidator { client.fetch(request) } catch (e: IOException) { return Result.CannotReach + } catch (e: IllegalArgumentException) { + return Result.CannotReach } // read the response stream to ensure the body is closed correctly. workaround for https://bugzilla.mozilla.org/show_bug.cgi?id=1603114