New issue
Advanced search Search tips

Issue 704901 link

Starred by 2 users

Issue metadata

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



Sign in to add a comment

Check supervised domain together with account type

Project Member Reported by rsorokin@chromium.org, Mar 24 2017

Issue description

realm locally-managed.localhost should work.
 
Labels: -Pri-3 M-61 Pri-2
Status: Started (was: Assigned)
Project Member

Comment 2 by bugdroid1@chromium.org, Aug 8 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/1f43178c5d1ba8d4b35841b21ad02d3ec75ed0f2

commit 1f43178c5d1ba8d4b35841b21ad02d3ec75ed0f2
Author: Roman Sorokin <rsorokin@chromium.org>
Date: Tue Aug 08 12:13:11 2017

Chromad: Do not allow supervised users on AD devices.

Moves checks about supervised user into *user_manager*.
Makes sure realm "locally-managed.localhost" is not
detected as supervised domain on Active Directory devices

Bug:  704901 
Test: ActiveDirectoryLoginTest.*
Change-Id: I01bd09e11d292c9ef9310e780b9774481a750209
Reviewed-on: https://chromium-review.googlesource.com/543342
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Roman Sorokin <rsorokin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#492600}
[modify] https://crrev.com/1f43178c5d1ba8d4b35841b21ad02d3ec75ed0f2/chrome/browser/chromeos/extensions/users_private/users_private_api.cc
[modify] https://crrev.com/1f43178c5d1ba8d4b35841b21ad02d3ec75ed0f2/chrome/browser/chromeos/login/existing_user_controller.cc
[modify] https://crrev.com/1f43178c5d1ba8d4b35841b21ad02d3ec75ed0f2/chrome/browser/chromeos/login/login_browsertest.cc
[modify] https://crrev.com/1f43178c5d1ba8d4b35841b21ad02d3ec75ed0f2/chrome/browser/chromeos/login/users/chrome_user_manager_impl.cc
[modify] https://crrev.com/1f43178c5d1ba8d4b35841b21ad02d3ec75ed0f2/chrome/browser/chromeos/login/users/fake_chrome_user_manager.cc
[modify] https://crrev.com/1f43178c5d1ba8d4b35841b21ad02d3ec75ed0f2/chrome/browser/chromeos/login/users/supervised_user_manager_impl.cc
[modify] https://crrev.com/1f43178c5d1ba8d4b35841b21ad02d3ec75ed0f2/chrome/browser/ui/webui/options/chromeos/accounts_options_handler.cc

Labels: -M-61 M-62
Status: Fixed (was: Started)
Status: Verified (was: Fixed)
Marked this as "Verified" since Supervised users can't be created or edited (https://support.google.com/chrome/answer/3463947) and lack of the issues with Chromad enrollment.

Sign in to add a comment