New issue
Advanced search Search tips

Issue 680670 link

Starred by 3 users

Issue metadata

Status: Verified
Owner:
Closed: Jan 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Missing normalization of user names causes user policy fetch to fail

Project Member Reported by tnagel@chromium.org, Jan 12 2017

Issue description

(some logging added by me)

The user is authenticated as Thiemo@chrome.lan, and the UPN is Thiemo@chrome.lan:

2017-01-12T21:35:48.420831+01:00 ERR authpolicyd[9233]: AuthenticateUser(Thiemo@chrome.lan)
2017-01-12T21:35:48.421223+01:00 INFO authpolicyd[9233]: Executing /usr/bin/kinit 'Thiemo@CHROME.LAN'
2017-01-12T21:35:48.635460+01:00 ERR authpolicyd[9233]: UPN: Thiemo@chrome.lan

However, when fetching user policy, thiemo@chrome.lan is used which fails:

2017-01-12T21:35:54.941546+01:00 ERR authpolicyd[9233]: FetchUserGpos(thiemo@chrome.lan)
2017-01-12T21:35:54.941553+01:00 ERR authpolicyd[9233]: No user logged in. Please call AuthenticateUser first.

Could you please also fix the "No user logged in." message?  That's a little misleading, better would be "User not logged in."
 
Cc: rsorokin@chromium.org
 Issue 677497  has been merged into this issue.

Comment 2 by tnagel@chromium.org, Jan 12 2017

I was just thinking that the switch to proper keys should fix that ... ;)

Comment 3 by tnagel@chromium.org, Jan 16 2017

Labels: -Pri-1 Pri-2

Comment 4 by tnagel@chromium.org, Jan 17 2017

Status: Started (was: Assigned)
Project Member

Comment 5 by bugdroid1@chromium.org, Jan 19 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform2/+/4ed5b612f6deee031afc2127193cd7f12a865e5a

commit 4ed5b612f6deee031afc2127193cd7f12a865e5a
Author: Roman Sorokin <rsorokin@chromium.org>
Date: Tue Jan 17 16:43:36 2017

authpolicy: Remove id by email.

BUG= chromium:680670 
TEST=manual

Change-Id: I10438c8dc336dfdd590ab58d4ed26bd43faf399f
Reviewed-on: https://chromium-review.googlesource.com/428870
Commit-Ready: Roman Sorokin <rsorokin@chromium.org>
Tested-by: Roman Sorokin <rsorokin@chromium.org>
Reviewed-by: Thiemo Nagel <tnagel@chromium.org>

[modify] https://crrev.com/4ed5b612f6deee031afc2127193cd7f12a865e5a/authpolicy/samba_interface.cc

Status: Fixed (was: Started)
Status: Verified (was: Fixed)
bulk Verify of older or not-user-facing Chromad bugs

Sign in to add a comment