[pve-devel] [PATCH 0/2] return cluster name on successful login if available

Dominik Csapak d.csapak at proxmox.com
Wed Nov 14 15:21:17 CET 2018


On 11/13/18 12:06 PM, Thomas Lamprecht wrote:
> This two patches fix #233 and #1358 which are somewhat related.
> 
> The, a bit dusty, #233 requested to have the cluster name returned when getting
> a ticket. This is useful information for API clients and thus I followed up and
> return this info, for successful authentications of users with the Sys.Audit
> privilege on '/'.
> 
> The second patch, for pve-manager, displays this information also in the UI, as
> requested by #1358, IMO also a sensible request. A lot of users have multiple
> clusters and can much easier differentiate between them with this patch.
> 
> Thomas Lamprecht (1):
>    fix #233: return cluster name on successful login
> 
>   PVE/API2/AccessControl.pm | 12 ++++++++++++
>   1 file changed, 12 insertions(+)
> 
> Thomas Lamprecht (1):
>    fix #1358: ui: show clustername besides 'Datacenter' node, if
>      available
> 
>   www/manager6/Workspace.js         |  2 ++
>   www/manager6/tree/ResourceTree.js | 17 +++++++++++++++++
>   2 files changed, 19 insertions(+)
> 

works, but maybe we want to remove the name on logout again?
if not, consider the patches reviewed & tested by me :)




More information about the pve-devel mailing list