profiles/iso-profiles
1
0
Fork 0
Code Issues 1 Pull requests Projects Releases Packages Wiki Activity Actions

Stop polkit-enabled programs from asking the user for a password in a live session #373

Closed
ghost wants to merge 0 commits from patch-nopasswd-live into master
pull from: patch-nopasswd-live
merge into: profiles:master
Conversation 13 Commits - Files changed -
ghost commented 2020-12-20 12:37:40 +00:00 (Migrated from gitlab2.manjaro.org)
Copy link

This is something that is very annoying. I get the question "what password do I use???" about 5 times a week. This MR simply removes the password for the "manjaro" user in a live session. This is already done for sudo, but not for polkit for some reason.

Additionally, I took the time to create the proper symlinks for NXD and some OEM stuff

Upstreamed from gitlab.com/garuda-linux/tools/iso-profiles/-@b72660f2a8

This is something that is very annoying. I get the question "what password do I use???" about 5 times a week. This MR simply removes the password for the "manjaro" user in a live session. This is already done for sudo, but not for polkit for some reason. Additionally, I took the time to create the proper symlinks for NXD and some OEM stuff Upstreamed from https://gitlab.com/garuda-linux/tools/iso-profiles/-/commit/b72660f2a8c842b4cd1d0100b967d9cf8d2ff977
ghost commented 2020-12-20 12:38:26 +00:00 (Migrated from gitlab2.manjaro.org)
Copy link

changed the description

changed the description
ghost commented 2020-12-20 12:47:40 +00:00 (Migrated from gitlab2.manjaro.org)
Copy link

added 2 commits

  • 08d3dd54 - Stop polkit-enabled programs from asking the user for a password in a live session
  • 70fd87b3 - Delete 50-max_user_watches.conf

Compare with previous version

added 2 commits <ul><li>08d3dd54 - Stop polkit-enabled programs from asking the user for a password in a live session</li><li>70fd87b3 - Delete 50-max_user_watches.conf</li></ul> [Compare with previous version](/profiles-and-settings/iso-profiles/-/merge_requests/180/diffs?diff_id=1828&start_sha=ad44a70dd983ed8cf857c140f04a060bb1088dfb)
ghost commented 2020-12-20 12:48:11 +00:00 (Migrated from gitlab2.manjaro.org)
Copy link

changed the description

changed the description
philm commented 2020-12-20 13:54:17 +00:00 (Migrated from gitlab2.manjaro.org)
Copy link

The issue with this is, that we can change the user name via our tools. So I rather would generate that polkit-1 rule on the fly via tools.

The issue with this is, that we can change the [user name](https://gitlab.manjaro.org/tools/development-tools/manjaro-tools/-/blob/master/lib/util-iso-image.sh#L306-309) via our tools. So I rather would generate that polkit-1 rule on the fly via **tools**.
ghost commented 2020-12-20 17:04:28 +00:00 (Migrated from gitlab2.manjaro.org)
Copy link

Alright, I'll take a look. Another alternative would be to just let UID=1000 be the condition, what do you think?

Alright, I'll take a look. Another alternative would be to just let UID=1000 be the condition, what do you think?
ghost commented 2020-12-21 09:08:59 +00:00 (Migrated from gitlab2.manjaro.org)
Copy link

Hm, manjaro-tools is a behemoth I really don't want to deal with right now, and it seems like polkit doesn't actually expose the UID. A hack would be to check if the user is in the autologin group, since that is the case by default.

Hm, manjaro-tools is a behemoth I really don't want to deal with right now, and it seems like polkit doesn't actually expose the UID. A hack would be to check if the user is in the autologin group, since that is the case by default.
ghost commented 2020-12-21 09:09:40 +00:00 (Migrated from gitlab2.manjaro.org)
Copy link

Maybe I'll take a look at pam and disable the password for the user in general 🤔

Maybe I'll take a look at pam and disable the password for the user in general :thinking:
philm commented 2020-12-21 09:23:01 +00:00 (Migrated from gitlab2.manjaro.org)
Copy link

mentioned in commit tools/development-tools/manjaro-tools@35f27fc4f3ad355143cff492c94afd1ee67e3675

mentioned in commit tools/development-tools/manjaro-tools@35f27fc4f3ad355143cff492c94afd1ee67e3675
philm commented 2020-12-21 09:24:26 +00:00 (Migrated from gitlab2.manjaro.org)
Copy link

mentioned in commit tools/development-tools/manjaro-tools@41242f4d5c4f259ca382e96d6bce5a3bac682693

mentioned in commit tools/development-tools/manjaro-tools@41242f4d5c4f259ca382e96d6bce5a3bac682693
ghost commented 2020-12-21 16:00:56 +00:00 (Migrated from gitlab2.manjaro.org)
Copy link

added 2 commits

  • fc7997cb - Fix missing symlinks
  • 1759cbf6 - Delete 50-max_user_watches.conf

Compare with previous version

added 2 commits <ul><li>fc7997cb - Fix missing symlinks</li><li>1759cbf6 - Delete 50-max_user_watches.conf</li></ul> [Compare with previous version](/profiles-and-settings/iso-profiles/-/merge_requests/180/diffs?diff_id=1831&start_sha=70fd87b3448973d93b5fc533a49bea808148424a)
ghost commented 2020-12-21 16:03:02 +00:00 (Migrated from gitlab2.manjaro.org)
Copy link

@philm Thanks for accepting this into manjaro-tools, I removed the polkit related changes from this merge request, so the rest should be ready for merging.

@philm Thanks for accepting this into manjaro-tools, I removed the polkit related changes from this merge request, so the rest should be ready for merging.
ghost commented 2022-02-25 09:58:13 +00:00 (Migrated from gitlab2.manjaro.org)
Copy link

This seems to have been merged - some house keeping would be to remove the MR - or am I mistaken somehow?

This seems to have been merged - some house keeping would be to remove the MR - or am I mistaken somehow?
yochananmarqos1 commented 2022-06-25 22:36:44 +00:00 (Migrated from gitlab2.manjaro.org)
Copy link

This is apparently already merged and quite old. Closing.

This is apparently already merged and quite old. Closing.
yochananmarqos1 (Migrated from gitlab2.manjaro.org) closed this pull request 2022-06-25 22:36:45 +00:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug

   
duplicate

   
enhancement

   
help wanted

   
invalid

   
question

   
wontfix
No labels
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
profiles/iso-profiles!373
No description provided.