[BUG] No longer loads http

foxriver76 / ioBroker-Visu-App

This is the issue tracker for the "ioBroker Visu" Android and iOS App

7 stars 0 forks source link

[BUG] No longer loads http #37

Closed udondan closed 7 months ago

udondan commented 7 months ago

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior

After the recent update to 1.0.32 the app no longer loads my vis, as it apparently now expects https.

The resource could not be loaded because the App Transport Security policy requires the use of a secure connection.

Expected Behavior

Load vis regardless of protocol.

Steps To Reproduce

No response

Which OS are you using?

iOS

OS Version

17.3.1

Device

iPhone 15 Pro Max

App Version

1.0.32

Anything else?

No response

foxriver76 commented 7 months ago

can you show the configuration, I cannot replicate this using of course simple http urls for local access

I will update to 17.3.1 and test with that too.

udondan commented 7 months ago

Right, it might also be related to last nights iOS update...

I have no special configuration, all default. Should have mentioned: I only use my local connection, no cloud.

udondan commented 7 months ago

I just tested on my wifes phone, she hasn't updated iOS yet. Same problem with iOS 17.2.1.

udondan commented 7 months ago

OK, I think I got it. When I use the IP of my raspberry, it works. When I use the hostname, which is resolved through my local DNS, it expects HTTPS.

But still, when loading the vis itself via IP, all resources which are referenced via http://rasp.pi:8089/... in my vis are not loaded, e.g. all flot diagrams.

foxriver76 commented 7 months ago

Hm okay, that maybe brings us one point closer, but not sure why the webview cares about this now..

The related App Transport Security settings have not changed from our side. So needs more investigation.

Strobelix80 commented 7 months ago

Same here Local dns fails Ip works

foxriver76 commented 7 months ago

Hm.. we have NSAllowsArbitraryLoads set to YES which should disable the whole ATS. However, according to the error message it fails due to the ATS policy - really strange.

foxriver76 commented 7 months ago

Due to some dependency issues it was not set correctly in the output, 1.0.33 will fix it and is currently being checked by Apple

udondan commented 7 months ago

Thanks for the quick fix!

foxriver76 commented 7 months ago

Does it work?

udondan commented 7 months ago

Review went fast. Confirming, 1.0.33 is working nicely.

foxriver76 commented 7 months ago

Yes Apple is way faster with reviews than google play most of the time 😬

thanks for the confirmation