Attribute Release view: display relying party's OrganizationDisplayName instead OrganizationName
Description
Environment
Activity
This has been changed for the next major release, the OrganizationDisplayName will be displayed instead of OrganizationName.
Talked to Scott on Slack, I plan on reverting the change for now but then scheduling for the next major release.
I think that the "display" name is the more appropriate name to use when displaying to users.
From the spec :
<OrganizationName> [One or More]
One or more language-qualified names that may or may not be suitable for human consumption.
<OrganizationDisplayName> [One or More]
One or more language-qualified names that are suitable for human consumption.
Since both OrganizationName and OrganizationDisplayName are required, I do not think it helps to use a conditional.
I would suggest doing nothing in a patch, this is something people need to do in their own templates.
No. We are using OrganizationDisplayName as RP name, so "SERVICE of SERVICE" is a little bit strange.
How about:
$rpOrganizationDisplayName == $serviceName ? $rpOrganizationName : $rpOrganizationDisplayName
in ./views/intercept/attribute-release.vm, line 25, the organization name of the Relying Party is fetched via
#set ($rpOrganizationName = $rpUIContext.organizationName)
it would be better to change this to
#set ($rpOrganizationName = $rpUIContext.organizationDisplayName)
because some federations like DFN-AAI or SWITCHaai don't polulate OrganizationName with a name suitable for human consumption