Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

T28600 unresponsive polkit dialog (quiet error) #205

Merged
merged 1 commit into from Jan 14, 2020

Conversation

mwleeds
Copy link
Contributor

@mwleeds mwleeds commented Jan 13, 2020

When a user who is not a sudoer clicks on another user in the Users
panel, if the panel is not unlocked it leads to an error message like
this:

Error retrieving app filter for user 'andrunko': Not allowed to query app filter data for user 1000

However it's expected behavior that the mct_manager_get_app_filter()
call will fail in this case, so check for such an error message and
clear it.

https://phabricator.endlessm.com/T28600

When a user who is not a sudoer clicks on another user in the Users
panel, if the panel is not unlocked it leads to an error message like
this:

Error retrieving app filter for user 'andrunko': Not allowed to query app filter data for user 1000

However it's expected behavior that the mct_manager_get_app_filter()
call will fail in this case, so check for such an error message and
clear it.

https://phabricator.endlessm.com/T28600
Copy link
Contributor

@pwithnall pwithnall left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@pwithnall pwithnall merged commit e9d2947 into master Jan 14, 2020
@pwithnall pwithnall deleted the T28600-parental-controls-quiet-error branch January 14, 2020 14:39
pwithnall added a commit to endlessm/malcontent that referenced this pull request Feb 27, 2020
This is a backport of
endlessm/gnome-control-center#205 from Endless’
gnome-control-center fork, applied there after the code was copied over
here.

Signed-off-by: Philip Withnall <withnall@endlessm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants