Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

App will not connect to server when adding account #170

Open
ghost opened this issue Dec 7, 2017 · 18 comments
Open

App will not connect to server when adding account #170

ghost opened this issue Dec 7, 2017 · 18 comments

Comments

@ghost
Copy link

ghost commented Dec 7, 2017

i have some issues with nextcloud sms, after trying the function reset pointer in the app, sync started to have issues with connecting to my server. i then tried to remove the account on my app and readd it, this is where the problem is, when readding the account i get a error message that the http:// server address is not reachable, i noticed that the error message says http:// is used eventhoug the app settings is set to https://
My server is not possible to reach with http at all, and if the app is ignoring the https:// ofcourse it would have issues connecting.

Would have been nice if you could fix this behavior

@nerzhul
Copy link
Owner

nerzhul commented Dec 7, 2017

Can you provide screen shots and android app version ?

@ghost
Copy link
Author

ghost commented Dec 10, 2017

Iam sorry m8, its not possible to upload screenshots from mobile to github, so no images for you.

Anyway the problem persist, iam going to dump databse regarding to ocsms and try if the problem might be server related. But i doubt it as its connecttion issues

@ghost
Copy link
Author

ghost commented Dec 10, 2017

screenshot_2017-12-10-13-35-16
screenshot_2017-12-10-13-35-49

@ghost
Copy link
Author

ghost commented Dec 10, 2017

Any progress?

@roydenyates
Copy link

roydenyates commented Dec 10, 2017

Can confirm CM13 (Android 6.0.1)
app ver 1.3.3.

I had it working and updated via f-Droid then saw notifications that the connection failed. However, if I opened app and manually synced there was no error. Yet the error notices persisted. Deleted account and here we are :-), no way back in.

PS: Nextcloud working just fine, so it seems to be an app error. Issue persists back to c 1.3.1. I think I upgraded from before 1.3.1.

@nerzhul
Copy link
Owner

nerzhul commented Dec 13, 2017

it's a difficult issue to track in the new and mandatory HTTP client i need time to track this, it seems nobody has java/android skills to help fixing it :(

@nerzhul
Copy link
Owner

nerzhul commented Dec 13, 2017

Also another fix has been pushed in 1.3.4 please re-test with it

@mourningsun75
Copy link

For me it is fixed

@nerzhul
Copy link
Owner

nerzhul commented Dec 16, 2017

@mourningsun75 nice to hear

@roydenyates
Copy link

roydenyates commented Dec 17, 2017 via email

@nerzhul
Copy link
Owner

nerzhul commented Dec 28, 2017

@prilly can you confirm ?

@barti
Copy link

barti commented Jan 4, 2018

Hi, even when using 1.3.4 I can still see the same issue: when trying to connect to https server and connection fails, the error message says that the http:// server address is not reachable. I guess, in my case connection error is related to self signed certificate feature request 172.
Anyway, the error with hard-coded "http" as a string is misleading and confusing.

@nerzhul
Copy link
Owner

nerzhul commented Jan 4, 2018

@barti exact i should fix the two issues. Sorry for the delay but i'm alone and there are rare PR on the app, and i'm working on many OSS projects :)

@JeromeSi
Copy link

Hi,
I've got the same problem in version 1.3.2, 1.3.3, 1.3.4.
With Android Studio, I make the apk with version 1.2.4 and this version work fine.
I've got lineageOS 14.1 (on a Moto G 4G, peregrine, android 7.1.2) with a nextcloud 11.0.6 and ocsms 1.11.4.
Thanks for the app.

@chrbayer
Copy link

Hi, it can not work with self signed certificates at the moment, it has to be allowed in the AndroidManifest.

in AndroidManifest.xml insert a new line after 37:android:allowBackup="true":
android:networkSecurityConfig="@xml/network_security_config"

and a new file res/xml/network_security_config.xml with his content:

This gives Android the permission to use self installed certificates, too :-)
Since the app crashes in my Emulator, I can not test, please someone with a working setup, make this small change :-)

And if it is working, I will purchase the app in the Store :-)

Best regards,
Christoph

@nerzhul
Copy link
Owner

nerzhul commented Jan 24, 2019

@chrbayer thanks for the tip, but the network_security_config.xml has no content in your post :)

also i'm not sure it works with a golang library

@chrbayer
Copy link

chrbayer commented Jan 24, 2019

@nerzhul This was the wrong issue, I wanted to insert my comment to #184, which I than did, and there is the missing file content :)

I'm not sure if it works for golang, too, but since it is some kind of safety feature, I would say, this permission should be independent from the used language...Nevertheless it is surely not wrong to tell the system that we want to use user certificates, so lose nothing in putting this in :)

@nerzhul
Copy link
Owner

nerzhul commented Jan 24, 2019

such permissions depends on the underlying OS but the apk permissions are for android SDK, and we don't use the SDK for HTTP requests, but i can add the options at least as metadata for play store

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants