25

Hi all bug reporting for your information. link

Problem details:

The Code - wifiManager.getConnectionInfo().getSSID()

The above code to returns the current SSID, it is returning the current SSID with extra quotations around it.

For eg. the SSID internet is returned as "internet".

This is only seen on Jelly bean 4.2 using device Nexus 7.

This bug is causing errors in our app as we compare the current SSID with the SSID that we are trying to connect too.

The code wifiManager.getScanResults(); however still returns all SSID's without extra quotation marks.


Lokesh Mehra
  • 545
  • 6
  • 16
Aiden Fry
  • 1,672
  • 3
  • 21
  • 45

5 Answers5

32

this is not a bug and behavior is correct as per documentation at http://developer.android.com/reference/android/net/wifi/WifiInfo.html#getSSID()

The so-called bug apparently was in pre 4.2 devices, because they didn't return it with "" enclosure.

Aiden's method looks good to me in the current state of confusion left by Android. However, being theoritically correct would just require

if (ssid.startsWith("\"") && ssid.endsWith("\"")){
             ssid = ssid.substring(1, ssid.length()-1);
}
kishu27
  • 3,140
  • 2
  • 26
  • 41
9

This regular expression is quite neat:

String ssid = wi.getSSID().replaceAll("^\"(.*)\"$", "$1");

Just for the notes

Edit °1 (as per question in the comment):

The issue that OP describes is, that on some devices the SSID returned by getSSID() is enclosed in "" whereas it is not on other devices. E.g. on some devices the SSID is "MY_WIFI" and on others it is MY_WIFI - or spoken in Java code: "\"MY_WIFI\"" and "MY_WIFI".

In order to to unify both results I proposed to remove the " at start and end - only there, because " is a legal character inside the SSID. In the regular expression above

^ means from start
$ means at end
\" means " (escaped)
.* means any number of characters
(...) means a capturing group, that can be referred by $1

So the whole expression means: replace "<something>" by <something> where $1 = <something>. If there is no " at end/start, the regular expression doesn't match and nothing is replaced.

See Java Pattern class for more details.

Trinimon
  • 13,839
  • 9
  • 44
  • 60
  • It *is* neat, but, it would help us to understand what it does, if you would add some explanations. The OP has no clear question as well. – msysmilu Nov 05 '15 at 14:21
6

For the mean time this is how I am getting around it, although its not great it will fix the issue.

 public String removeQuotationsInCurrentSSIDForJellyBean(String ssid){
     int deviceVersion= Build.VERSION.SDK_INT;

     if (deviceVersion >= 17){
         if (ssid.startsWith("\"") && ssid.endsWith("\"")){
             ssid = ssid.substring(1, ssid.length()-1);
         }
     }

     return ssid;

 }
Aiden Fry
  • 1,672
  • 3
  • 21
  • 45
4

Two very simple variants:

string = string.replaceAll("^\" | \"$", "");

and

string = string.substring(1, string.length() - 1);
Oleksandr Kravchuk
  • 5,963
  • 1
  • 20
  • 31
  • 2
    be careful with these as it is possible for valid SSID's to have quotes within them ie. "my "freeking" wifi". Also the second varient is great but you need the check mentioned in my answer to make sure you don't mess up anything else. – Aiden Fry Nov 28 '12 at 11:54
  • @AidenFry I don't think the regex would affect the example you provided, since it only check the starting char and ending char (with ^ and $). The function would only return String with quotes or hex digits according to the doc, so it only eliminates quotes rounded by the function but would keep the quotes of the original String. The problem with the regex is it should not have **empty space** after/before the quotes. e.g. `("^\"|\"$")` – Dante Y Oct 17 '13 at 15:01
  • It is neat, but, it would help us to understand what it does, if you would add some explanations. The OP has no clear question as well. – msysmilu Nov 05 '15 at 14:21
0

Faced the same problem! Used this technique which is backwards compatible:

if (suppliedSSID.equals(connectionInfo.getSSID()) || ("\"" + suppliedSSID + "\"").equals(connectionInfo.getSSID()) { DO SOMETHING }

user357614
  • 111
  • 1
  • 5