Differential D6003

Tell user to unlock his session only
ClosedPublic
Actions

Authored by urkud on May 28 2017, 11:18 PM.

Details

Reviewers
davidedmundson
Group Reviewers
Plasma
Summary

In default configuration, loginctl unlock-sessions requires root password, while unlocking another session of the same user doesn't.

Diff Detail

Repository
R133 KScreenLocker
Lint
Lint Skipped
Unit
Unit Tests Skipped
urkud created this revision.May 28 2017, 11:18 PM
Restricted Application added a project: Plasma. · View Herald TranscriptMay 28 2017, 11:18 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
davidedmundson accepted this revision.May 29 2017, 11:43 AM
davidedmundson added a subscriber: davidedmundson.

Can you confirm you've tested this.

Do you have commit access?

This revision is now accepted and ready to land.May 29 2017, 11:43 AM
urkud added a comment.May 29 2017, 2:32 PM
In D6003#112523, @davidedmundson wrote:

Can you confirm you've tested this.

I've tested that (a) it shows new message; (b) new command works without root password.

I've tried to run all autotests, but some of them fail both with and without this patch.
Is there any guide on how to run them correctly?

Do you have commit access?

Yes, I have.

Revision Contents
Changeset List

PathPackages
Mabstractlocker.cpp (5 lines)

Diff 14920

View Options

abstractlocker.cpp

Loading...