Github user nir-sopher commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/398#discussion_r109313737
--- Diff: traffic_ops/app/lib/API/User.pm ---
@@ -277,6 +285,7 @@ sub current {
@data, {
"id" => "0",
"username" => $current_username,
+ "tenantId" =>
$self->current_user_tenant(),
--- End diff --
I assume the tenancy of an LDAP user should be set by the operator. He
might want it to be the single tenant in the system (if he has only one
tenant). He might want it to be the "root" tenant. And he might want it to be
"undef" which is a powerless tenant (such like a read-only user).
If you agree with the above I think we need to provide a way to specify the
"LDAP" users' tenant when the tenancy feature is first activated. Until then
(and if not set) an "undef" is assumed.
What would you say about adding a property in the tenants table, allowing
to mark the "LDAP" tenant?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
