Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(467)

Issue 2753243003: Use sudo when starting session as normal user (Closed)

Created:
3 years, 9 months ago by rkjnsn
Modified:
3 years, 9 months ago
Reviewers:
Sergey Ulanov, Jamie
CC:
chromium-reviews, chromoting-reviews_chromium.org
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Use sudo when starting session as normal user The PAM wrapper currently only works when started as root, so use sudo for sessions started by the user for now. BUG=702307 Review-Url: https://codereview.chromium.org/2753243003 Cr-Commit-Position: refs/heads/master@{#457893} Committed: https://chromium.googlesource.com/chromium/src/+/a35ddcb44aeea4436fa75bb6419e3b428377642c

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -1 line) Patch
M remoting/host/installer/linux/debian/chrome-remote-desktop.init View 1 chunk +7 lines, -1 line 0 comments Download

Messages

Total messages: 13 (8 generated)
rkjnsn
3 years, 9 months ago (2017-03-17 19:55:16 UTC) #4
Jamie
lgtm
3 years, 9 months ago (2017-03-17 21:40:05 UTC) #7
Sergey Ulanov
lgtm
3 years, 9 months ago (2017-03-17 22:05:50 UTC) #8
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2753243003/1
3 years, 9 months ago (2017-03-17 22:10:02 UTC) #10
commit-bot: I haz the power
3 years, 9 months ago (2017-03-17 22:24:04 UTC) #13
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/a35ddcb44aeea4436fa75bb6419e...

Powered by Google App Engine
This is Rietveld 408576698