send a signal over dbus when the wrong password is provided
ClosedPublic

Authored by mart on Feb 25 2019, 4:11 PM.

Details

Summary

On Plasma Mobile we can't actually show a qwidgets based dialog
when the password is wrong, and preferably not any additional window over the $
this adds a signal to the kded when it fails to authenticate,
the handler will forget such password so that the password field will appear again and the kcm shows an error message when this happens

Test Plan

tried to connect with wrong password

Diff Detail

Repository
R116 Plasma Network Management Applet
Branch
phab/errorSignal
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 8828
Build 8846: arc lint + arc unit
mart created this revision.Feb 25 2019, 4:11 PM
Restricted Application added a project: Plasma. · View Herald TranscriptFeb 25 2019, 4:11 PM
Restricted Application added 1 blocking reviewer(s): jgrulich. · View Herald Transcript
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
mart requested review of this revision.Feb 25 2019, 4:11 PM
mart added a reviewer: bshah.Feb 25 2019, 4:12 PM
jgrulich accepted this revision.Mar 2 2019, 8:14 PM

Good idea to remove the failed connection.

kded/secretagent.cpp
412

I'm not sure this assumption is currect, it's not that the password is wrong all the time, NM might be just asking for additional passwords or for passwords which are marked not to be stored (e.g. VPN, or WPA2 EAP). But I guess for your usecase we can leave it as it is, we just need to make sure that Configuration::showPasswordDialog() will be true only on plasma-mobile.

libs/handler.h
107

void secretAgentError(const QString &connectionPath, const QString &message)

so we know on first look what are these params?

This revision is now accepted and ready to land.Mar 2 2019, 8:14 PM
This revision was automatically updated to reflect the committed changes.