Skip to content

metadata maker is generating empty URI in ServiceName element #345

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Closed
rhoerbe opened this issue Jun 24, 2016 · 6 comments
Closed

metadata maker is generating empty URI in ServiceName element #345

rhoerbe opened this issue Jun 24, 2016 · 6 comments

Comments

@rhoerbe
Copy link
Contributor

rhoerbe commented Jun 24, 2016

Pysaml2 is generating an <--ns0:ServiceName xml:lang="en"/> element for the AttributeConsumingService, which is XML schema valid, but not allowed according to SAML core:

1.3.2 URI Values
..
Unless otherwise indicated in this specification, all URI reference values used within SAML-defined elements or attributes MUST consist of at least one non-whitespace character ..

@rhoerbe
Copy link
Contributor Author

rhoerbe commented Jun 24, 2016

Ah, my bad, this is not an localizedURIType, it is a localizedNameType. I got confused by the OpenSAML error message

@rhoerbe rhoerbe closed this as completed Jun 24, 2016
@rhoerbe
Copy link
Contributor Author

rhoerbe commented Jun 24, 2016

just learned that strings cannot be blank, like URIs:
SAML core 1.3.1 String Values:
... Unless otherwise noted in this specification or particular profiles, all strings in SAML messages MUST consist of at least one non-whitespace character

@rhoerbe rhoerbe reopened this Jun 24, 2016
@jkakavas jkakavas added the bug label Jul 26, 2017
@peppelinux
Copy link
Member

Hi @rhoerbe, I can't reproduce this in my asset, I have a SP with a correct valued ACS.
Can give more information about how to reproduce this?

@rhoerbe
Copy link
Contributor Author

rhoerbe commented Sep 6, 2020

I passed on my SATOSA-related projects, so I have no dev env available in the time since. But if pysaml now is creating a proper string value, or no ServiceName at all, then the problem should be fixed.

@peppelinux
Copy link
Member

Thank you for ack. I close this for now because It looks good to me, if someone Will update this thread I'll open It again

👍

@melanger
Copy link

@peppelinux This was probably caused by a missing name under the sp key in config, such as in the example SATOSA SAML2 backend config. I created IdentityPython/SATOSA#393 to address that.

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

No branches or pull requests

4 participants