Ticket was cloned from Red Hat Bugzilla (product Red Hat Enterprise Linux 7): Bug 1272217
Description of problem:
This is an enhancement request to add the ability to use some characters other
than just letters and numbers for SP names. The purpose here is to allow the
same character set allowed for hostnames.
This is an example of not being able to use a hostname:
ipsilon-client-install --saml-auth /secure \
> --saml-sp-name $(hostname -s) \
> --saml-idp-url https://rhel7-2.example.com/idp
Failed to register SP with IDP!
Error: ['Invalid Provider ID: Name must contain only numbers and letters']
'Invalid Provider ID: Name must contain only numbers and letters'
Version-Release number of selected component (if applicable):
Steps to Reproduce:
1. ipsilon-client-install --saml-sp-name $(hostname)
if enhanced to support more characters, I would like to see this work.
milestone: N/A =>
patch_available: => 0
milestone: => 1.3
I looked into this a while back, the SP name is used in two different contexts, as a key in the database and as path component in a URL. I suspect it's the later use as a URL path component which prompted the character restrictions. If we URL encode/decode the name when it's used in a URL then I think we can remove the restriction. I don't think it's use as a database key creates a problem.
For period and dash, we don't need any special URL encoding/decoding.
The patch at https://pagure.io/ipsilon/pull-request/82 adds those two characters, so that we allow RFC-952 compatible hostnames.
component: Documentation => SAML
owner: => puiterwijk
patch_available: 0 => 1
status: new => accepted
This has been fixed with 9035f18
resolution: => fixed
status: accepted => closed
Turns out that dots and hyphens are a problem in some cases.
Reopening this ticket to track the fixing of this.
resolution: fixed =>
status: closed => reopened
This has been fixed in https://pagure.io/ipsilon/c/3452be826f4ba256a77995cf42bbb97a3b0b39bc.
resolution: => fixed
status: reopened => closed
Metadata Update from @puiterwijk:
- Issue assigned to puiterwijk
- Issue set to the milestone: 1.3
to comment on this ticket.