nerzhul / ncsms-android

repository for the ncsms on Android
95 stars 38 forks source link

unable to connect to local OwnCloud instance #120

Closed nate2014jatc closed 7 years ago

nate2014jatc commented 7 years ago
### Steps to reproduce 1. Add new account 2. get failure to connect message ### Expected behaviour Tell us what should happen Connects and syncs ### Actual behaviour Tell us what happens instead Fails to connect ### Server configuration **ownCloud version:** 9.1.3 **PHP version:** 7.0.8 **HTTPd server:** Apache2.4.18 **HTTPS:** yes ### Client configuration **Android version:** 7.1.1 **Phone:** Nexus 6P **ownCloud SMS app version:** 0.23.0 ### Logs ``` Insert your log here ``` ![screenshot_20161215-014549](https://cloud.githubusercontent.com/assets/11836893/21217210/4c5dda54-c268-11e6-84e1-c0a4a20f2a5a.png) ![screenshot_20161215-014800](https://cloud.githubusercontent.com/assets/11836893/21217289/afd9dcc2-c268-11e6-8c0e-739669dc025a.png)
nerzhul commented 7 years ago

Hello, did you use a self signed certificate ? Where does the app come from ?

EvilBob01 commented 7 years ago

I have the same issue. I'm running nextcloud11.0.0 I'm using a Self signed certificate. and I got the app V0.23.0 from F-droid. I've tried using http:// from the drop down to mitigate any ssl cert issues. and the message I receive is "UNK"

Any suggestions?

nate2014jatc commented 7 years ago

I'm using a self signed cert, and I got the app from f-droid.

Trying to make a https connection On Thu, Dec 15, 2016, 10:43 AM EvilBob01 notifications@github.com wrote:

I have the same issue. I'm running nextcloud11.0.0 I'm using a Self signed certificate. and I got the app V0.23.0 from F-droid. I've tried using http:// from the drop down to mitigate any ssl cert issues. and the message I receive is "UNK"

Any suggestions?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/nerzhul/ownCloud-SMS-App/issues/120#issuecomment-267392659, or mute the thread https://github.com/notifications/unsubscribe-auth/ALSd3fwSUtgqItOsEt6JmNVjk2LiqBH6ks5rIXxPgaJpZM4LN2Q2 .

nerzhul commented 7 years ago

The problem is the F-Droid build which is not official and doesn't include the patched owncloud library for connecting into https, sorry

EvilBob01 commented 7 years ago

That's good to know. I appreciate you replying it saves me so much time troubleshooting.

I wasn't aware that the F-Droid build was not official. It's mentioned here https://github.com/nerzhul/ownCloud-SMS-App/README.md

If I grab the source on github and build it will that include the patched OC library? Or is Google Play the only supported / recomended build?

Thanks again for your time and work on this project?

On 12/15/2016 02:44 PM, nerzhul wrote:

The problem is the F-Droid build which is not official and doesn't include the patched owncloud library for connecting into https, sorry

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/nerzhul/ownCloud-SMS-App/issues/120#issuecomment-267424289, or mute the thread https://github.com/notifications/unsubscribe-auth/AGZoM2-_ycz5mHVKEWMPuQo4mKRsDTcJks5rIZiTgaJpZM4LN2Q2.

sarnold commented 7 years ago

Could you be a little more specific please? What patch and where is it? Does it exist in the owncloud library repo or are you keeping it secret? Open source is kinda the opposite of secret-sauce patches...

nerzhul commented 7 years ago

The patch is only removal a security on the lib on self signed certificates because lib isn't very aware to this and doesn't resend a return back to client to allow him to allow the self signes certificates

nerzhul commented 7 years ago

owncloud library is now dropped from our app, it's not necessary, now i need to ensure self signed certificates works. They are not working out of the box

nerzhul commented 7 years ago

This is now fixed by 1257de1a0a6d5f768b406b8aa9fb9ced5a92ab40 owncloud library was removed and the client is now very simple, not up to date atm because we are using legacy apache commons http client but we should move to recent later.

Can you build 0.23.2 release and test it ?

nerzhul commented 7 years ago

This is definitively fixed and we didn't use owncloud library, less memory usage and better support for SSL (not the best but works everywhere)