search

PantryParty / pantry_party

Apache License 2.0
70 stars 8 forks source link

A network error was encountered. Please check your internet connection and try again #51

Closed totti4ever closed 4 years ago

totti4ever commented 4 years ago

After inserting the API key. The given URL is a local only, but reachable from my Android.

arjes commented 4 years ago

Thanks for the report.

There have been a few reports of this on reddit. That error indicates the app had it's connecting terminated for odd reasons (status code 0). Are you using http or https?

totti4ever commented 4 years ago

http

mwolfaardt commented 4 years ago

same with mine

arjes commented 4 years ago

I was able to reproduce this in a simulator on Android 10. It looks like at some point android started requiring https connections from apps. I'm going to review the android docs and see if I can do anything to allow these connections.

If it is possible for you to install a LetsEncrypt cert, that should clear it up immediately.

totti4ever commented 4 years ago

Okay, I just figured out that I actually had setup grocy to listen not only on my local .home domain per http but also on my external domain via https, because otherwise I couldn't use the barcode scanner/camera (Chrome restriction). I had already secured it by an IP filter, so only local IPs are allowed.

So I'm fine now, thanks!

P.S.: Think about donation instead of ads (how much money do you get from ads anyway? I suppose, donating 5€ is worth heaps installations with ads enabled?

arjes commented 4 years ago

I've pushed a new release for Android which should allow you to connect to non-http endpoints. Let me know if that solves your problem.

As for the ads vs donations option. I honestly don't know the long term CPM of ads or the frequency of donations. I'm about $150 into the project, and apple requires a continuing subscription to continue to publish the app. I would be surprised if I ever recoup the cost, but I figure I'll save that much in groceries every year.

I only included the ads because it is better that I start with ads, then add them later.

mwolfaardt commented 4 years ago

I see in the appstore the latest version is for 4 May, will your changes become available via the android app store? Also have a look at barcode buddy, they setup integration by scanning a barcode, and it works great

arjes commented 4 years ago

@mwolfaardt that new version was out yesterday, based on your comment though I'm guessing you don't see it yet. That is odd I did not stage the rollout so it should be available to everyone.

mwolfaardt commented 4 years ago

Thanks, I've tried again today and it's working

arjes commented 4 years ago

@mwolfaardt Thanks for reporting back. I'm going to close this but feel free to reopen it if you have further trouble

renannprado commented 4 years ago

@arjes I face the same error while connecting to my grocy API.

I've installed my home CA in my phone and other apps such as browsers and bitwarden stopped complaining after that, but with Pantry Party I was forced to use HTTP instead of HTTPS in order to use the app.

Would be nice if you can put the actual error message that happened so that it's easier for to report. At the moment I believe that the app is not trusting, for some reason, the CAs that were added by users.

arjes commented 4 years ago

@renannprado Sorry about that. Unfortunately since this is a Nativescript app I have limited access to some diagnostics and all network level errors tend to show up as simply code 0.

I'll try and re-create it locally and see if the error body is more helpful.