Review Request 124140: Ported over to Netty and added SSL support

Vineet Garg grg.vineet at gmail.com
Sun Jun 21 12:41:04 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124140/
-----------------------------------------------------------

Review request for kdeconnect and Albert Vaca Cintora.


Repository: kdeconnect-android


Description
-------

* Since MINA was tying hands in adding SSL, ported over Netty which has a good, clean and easy interface and a larger user and developer base.
* Added support to setup links on SSL
* Links automatically removed if wrong certificate is sent
* Shows keys based on hash of certificate to check right certificates are received
* Added a preference to use SSL, as it is experienced to cause high CPU usage on devices.
* Corrected unit tests, LanLinkProvider is removed as it is nearly impossible to write it with current design. Will find a way


Diffs
-----

  build.gradle 6520c1c 
  res/layout/activity_pair.xml 6b73edc 
  res/values/strings.xml c128342 
  res/xml/general_preferences.xml 82ab0ca 
  src/org/kde/kdeconnect/Backends/BaseLink.java 579a7af 
  src/org/kde/kdeconnect/Backends/LanBackend/LanLink.java 5994142 
  src/org/kde/kdeconnect/Backends/LanBackend/LanLinkProvider.java ae26958 
  src/org/kde/kdeconnect/Backends/LoopbackBackend/LoopbackLink.java add92f8 
  src/org/kde/kdeconnect/Backends/LoopbackBackend/LoopbackLinkProvider.java bd9c41b 
  src/org/kde/kdeconnect/BackgroundService.java 5e3d8c2 
  src/org/kde/kdeconnect/Device.java 3262aef 
  src/org/kde/kdeconnect/Helpers/SecurityHelpers/RsaHelper.java PRE-CREATION 
  src/org/kde/kdeconnect/Helpers/SecurityHelpers/SslHelper.java PRE-CREATION 
  src/org/kde/kdeconnect/NetworkPackage.java f62b5d0 
  src/org/kde/kdeconnect/UserInterface/MainSettingsActivity.java 1b0e230 
  src/org/kde/kdeconnect/UserInterface/PairActivity.java 7a45751 
  tests/org/kde/kdeconnect/DeviceTest.java 5d3383d 
  tests/org/kde/kdeconnect/LanLinkProviderTest.java fe7863a 
  tests/org/kde/kdeconnect/LanLinkTest.java d3d94c9 
  tests/org/kde/kdeconnect/NetworkPackageTest.java a21114e 

Diff: https://git.reviewboard.kde.org/r/124140/diff/


Testing
-------

Tesed on some device with where both supports SSL, also with PC where ssl is not supported, working fine. Need a little bit more testing with more devices.


Thanks,

Vineet Garg

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20150621/9ee561df/attachment.html>


More information about the KDEConnect mailing list