openconnect: support MFA
Needs RevisionPublic

Authored by scarpino on Jan 8 2018, 10:43 PM.

Details

Reviewers
jgrulich
Summary

support MFA authentication in openconnect

Test Plan

build and tested

Diff Detail

Lint
Lint Skipped
Unit
Unit Tests Skipped
scarpino created this revision.Jan 8 2018, 10:43 PM
Restricted Application added a project: Plasma. · View Herald TranscriptJan 8 2018, 10:43 PM
scarpino requested review of this revision.Jan 8 2018, 10:43 PM
scarpino updated this revision to Diff 24997.Jan 9 2018, 10:49 AM

Store tokenSecret as secret.

jgrulich requested changes to this revision.Jan 15 2018, 8:21 AM

Could you also add support for RSA SecurID so support for tokens is complete? I'm also not sure whether with this the support is complete, because you need to also add this to authentication dialog if I'm not mistaken.

See https://bugs.kde.org/show_bug.cgi?id=338650.

vpn/openconnect/openconnectwidget.cpp
111

Missing bracket at the end.

This revision now requires changes to proceed.Jan 15 2018, 8:21 AM