Review Request 125266: Authentication with certificates / Make MRU/MTU editable / Dialog fix

René Fürst rene at fuerst.biz
Tue Feb 2 22:17:23 UTC 2016


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

(Updated Feb. 2, 2016, 10:17 p.m.)


Status
------

This change has been marked as submitted.


Review request for Network Management and Plasma.


Changes
-------

Submitted with commit b4c7cf28c3fea1ce54ca86d7d65494cc861b69bf by Lamarque V. Souza on behalf of Rene Furst to branch master.


Repository: plasma-nm


Description
-------

Authentication with certificates / Make MRU/MTU editable / Dialog fix


Diffs
-----

  vpn/l2tp/l2tp.ui bf35d02 
  vpn/l2tp/l2tpppp.ui 3c70165 
  vpn/l2tp/l2tppppwidget.cpp ffe2c2b 
  vpn/l2tp/l2tpwidget.h a161b53 
  vpn/l2tp/l2tpwidget.cpp a4ff42f 
  vpn/l2tp/nm-l2tp-service.h ac2ecc9 

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


Testing
-------

This patch contains 3 things:
1) Add authentication with certificates
2) Make MRU/MTU editable
3) Fix an issue where PPP/Advanced settings were lost when the dialogs were not opened

The base for 1) and 2) was added to NetworkManager-l2tp here https://github.com/frenetic1/NetworkManager-l2tp/commit/8103cf09e2cda19d701a48331eba069ff4c8e82c


File Attachments
----------------

Patch without indent changes
  https://git.reviewboard.kde.org/media/uploaded/files/2015/12/16/e492aabf-c098-4ea7-bff0-30a13cc3fdda__cert_mru_mtu_dialog_fix4_noindent.diff


Thanks,

René Fürst

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160202/93135e61/attachment-0001.html>


More information about the Plasma-devel mailing list