MapboxAccess does not work on Android

[mapbox]

MapboxAccess does not work on Android

[wilhelmberg]

Commenting validation and hard coding the token works:

Although there seem to be other problems too as buildings and their styles are still missing.

EDIT
After commenting ValidateMapboxAccessFile() and hard coding the token the rest works as expected:

[david-rhodes]

@BergWerkGIS Yeah, since I don't have Android I was unable to test myself. I thought that @MateoV had tested to ensure everything was working as expected, though.

The actual loading of the text file is done with WWW, but as you mentioned ValidateMapboxAccessFile fails because the "file is not found" on Android. I think we should just #if !UNITY_ANDROID the contents of this method and do some extra error checking in IEnumerator LoadMapboxAccess().

[david-rhodes]

@BergWerkGIS do you know if the directory is not able to be found, or if the file does not exist?

if (!Directory.Exists(Application.streamingAssetsPath) || !File.Exists(_accessPath)) {
    throw new InvalidTokenException("Please configure your access token in the menu!");
}

[david-rhodes]

Regarding Problem 2, we may have to hold a queue of requests until the async read is done, then fire them off. This seems like a simple fix, but I won't be able to test. Do you have a better idea?

[wilhelmberg]

do you know if the directory is not able to be found, or if the file does not exist?

I think Directory.Exists() and File.Exists() don't work with the url that points into the jar file.

Regarding Problem 2, we may have to hold a queue of requests until the async read is done, then fire them off. This seems like a simple fix, but I won't be able to test. Do you have a better idea?

I would prefer to somehow make loading of the token sync.
Otherwise all code that requests anything would have to check and wait till the token is available.

Will have a look tomorrow.

How about not using IEnumerator and yield but something like:

void LoadMapboxAccess() {
    var request = new WWW(_accessPath);
    while (!request.isDone) { Thread.Sleep(10); }
    AccessToken = request.text;
}

[david-rhodes]

How about not using IEnumerator and yield?

I never thought to use WWW like that, but should be possible. We would have to include a timeout or error handling to prevent freezing the entire application. Android is so fragmented, you never know what device or OS this will not work for 😏.

[isiyu]

@david-rhodes and I tried a couple of solutions wanted to get your thoughts on these (sorry the diffs aren't super helpful here) Tested on and Android device and the styles and buildings demo loads. https://mapbox.slack.com/files/isiyu/F53553D55/image_uploaded_from_ios.mp4

@BergWerkGIS wanted to get your thoughts on which solution to go with for now

option A: cache requests 3ffc117

What the logs look like for a successful run, with a valid token:

• https://docs.google.com/a/mapbox.com/document/d/1zvyOtyqmvtUpcRL_FHoh_Wv_e-s1dqaiXFSmz_CAZTE/edit?usp=sharing

option B: thread sleep 5125bce

with no token:

[david-rhodes]

For clarity, Option B works. Please note: #if !_UNITY_ANDROID should be #if !UNITY_ANDROID. Testing without a token was just to see what exception was thrown. The correct InvalidTokenException was thrown. 👍

@BergWerkGIS this was your suggestion, and it seems to work as intended. Any idea why your implementation did not work earlier today? In any case, we would probably want to implement a timeout for the while loop and throw an exception in the event the WWW never finishes.

As mentioned in slack, Option A is likely "safer," but it is less efficient and does not currently allow cached requests to be cancelled (but this is seriously an edge case). We can fix this, but I'd like to know your thoughts, first.

[wilhelmberg]

@BergWerkGIS this was your suggestion, and it seems to work as intended. Any idea why your implementation did not work earlier today?

No idea, maybe some error I missed during deploy and an old version was used on the device.
Works today. 🎉

I'd like to know your thoughts, first.

I'd prefer Option B to caching at that level.

---

My PR for MapboxAccess.cs, including:

• Android fix
• timeout for LoadMapboxAccess()
• UWP support
• ServerCertificateValidationCallback to make HttpWebRequest on Mono work with HTTPS requests

[david-rhodes]

Fixed: #26