We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
perl-Net-SAML2/lib/Net/SAML2/SP.pm
Line 351 in 305c587
As reported by @wora55 there is a (minor) bug in metadata, where the Organization url is incorrectly defined, because it used $self->url.
It should likely be
$x->OrganizationURL( $md, { 'xml:lang' => 'en' }, $self->org_url || $self->url
and add another (optional) line for the object so existing modules would not have to be changed
https://github.com/perl-net-saml2/perl-Net-SAML2/blob/master/lib/Net/SAML2/SP.pm#L83
has 'org_contact' => (isa => 'Str', is => 'ro', required => 1); has 'org_url' => (isa => 'Str', is => 'ro', required => 0);
The text was updated successfully, but these errors were encountered:
Fixes perl-net-saml2#18 - SP metatdata org_url
27d4d19
9944c41
Merge pull request #20 from timlegge/issue18
5377ecd
Fixes #18 - SP metatdata org_url
timlegge
No branches or pull requests
perl-Net-SAML2/lib/Net/SAML2/SP.pm
Line 351 in 305c587
As reported by @wora55 there is a (minor) bug in metadata, where the Organization url is incorrectly defined, because it used $self->url.
It should likely be
$x->OrganizationURL(
$md,
{
'xml:lang' => 'en' },
$self->org_url || $self->url
and add another (optional) line for the object so existing modules would not have to be changed
https://github.com/perl-net-saml2/perl-Net-SAML2/blob/master/lib/Net/SAML2/SP.pm#L83
has 'org_contact' => (isa => 'Str', is => 'ro', required => 1);
has 'org_url' => (isa => 'Str', is => 'ro', required => 0);
The text was updated successfully, but these errors were encountered: