Skip to content

Commit 7aff3fa

Browse files
committed
Remove @ validation from network settings
Was a PR to completely remove space and @ validation (#985), but I can see people easily pressing space without meaning to and not being able to see what is wrong. If someone makes a good reason for it to be needed I will remove it, but for now just removing @ so people can do basic auth.
1 parent 6bb8745 commit 7aff3fa

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

app/src/main/java/github/daneren2005/dsub/fragments/SettingsFragment.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -803,7 +803,7 @@ public boolean onPreferenceChange(Preference preference, Object value) {
803803
try {
804804
String url = (String) value;
805805
new URL(url);
806-
if (url.contains(" ") || url.contains("@")) {
806+
if (url.contains(" ")) {
807807
throw new Exception();
808808
}
809809
} catch (Exception x) {
@@ -824,7 +824,7 @@ public boolean onPreferenceChange(Preference preference, Object value) {
824824
}
825825

826826
new URL(url);
827-
if (url.contains(" ") || url.contains("@")) {
827+
if (url.contains(" ")) {
828828
throw new Exception();
829829
}
830830
} catch (Exception x) {

0 commit comments

Comments
 (0)