<div dir="ltr">Hi Vineet,<br><br>First of all, please push soon the refactor you made as part of your first milestone. I think I gave you a "Ship it" already. According to the schedule, it should have been merged before May 30th, and I can see it's not done yet.<div><br></div><div>The next step according to the roadmap is: "15 June -> SslLink link implemented on Android, code-reviewed and merged." How are we tracking on that goal? It's okay to me if you decide to work on the tests first, but please make sure that you don't take longer than initially planned for both things. I already commented on the code review you sent with the tests. It's very nice to finally have a testing framework in place for KDE Connect in Android :D</div><div><br></div><div>About the SSL link, I saw the changes in your Github repo. I would like to know what are the reasons for re-using the LanLink instead of creating a new type of Link. Is it better this way? Why?</div><div><br></div><div>Finally, about testing with two phones, it is definitely going to be better than using two emulators, but actually the emulator should work as well. I have two phones, so I should be able to test your changes, but I haven't had time to do it yet and you shouldn't rely on me having free time to test it. Keep me posted on when you think you will be able to go back to working on this (either getting a second phone or making the emulator work).<br></div><div><br></div><div>Happy hacking!</div><div><br></div><div>Albert</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jun 5, 2015 at 4:41 AM, vineet garg <span dir="ltr"><<a href="mailto:grgvineet@gmail.com" target="_blank">grgvineet@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div><div>Hi, <br>I have been working with ssl with mina for past few days. While testing I realized that there are some compatibility issues with mina that it is not working in emulator properly. The same code when tested with openssl cli is working on physical device but not working in emulator correctly.<br><br></div>Due to simplicity of mina to handle ssl via filter, I was able to add ssl filter in session within LanLink, both device if supports ssl, ssl will be started. If even one device is old, the ssl won't be started and normal link proceeds. Have a look of the code, available at "<a href="https://github.com/grgvineet/kdeconnect-android/tree/sslinlanlink" target="_blank">https://github.com/grgvineet/kdeconnect-android/tree/sslinlanlink</a>", also test it whether it works on two physical device if possible.<br></div><div>Currently device trust all certificates in all connections. Method to check for remote device with its certificate is added but not tested yet and not in use currently.<br></div><div><br></div>Since I have only single device, I am not able to test it properly. Planning to buy new phone, but that is only available Thursday in flash sale at Amazon. So I have to wait for a week plus the time it will take to reach home.<br><br></div>I am thinking to preempt this for some time and get my hands dirty on Android testing framework. What do you suggest ?<span class="HOEnZb"><font color="#888888"><br><br></font></span></div><span class="HOEnZb"><font color="#888888">Vineet<br><div><div><br> <br></div></div></font></span></div>
</blockquote></div><br></div>