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
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
17a9d00
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