Skip to content
New issue

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

Issues with XML and html/template #1

Closed
enrichman opened this issue Dec 7, 2018 · 1 comment
Closed

Issues with XML and html/template #1

enrichman opened this issue Dec 7, 2018 · 1 comment
Labels
bug Something isn't working

Comments

@enrichman
Copy link

SP Metadata are not correctly parsed since the html/template package is not meant for XML templating (see: golang/go#3133 )
It results in having the first line escaped in

<?xml version="1.0"?>

I would suggest to delegate the SAML XML logic to a more complete and robust library implementation, such as https://github.com/RobotsAndPencils/go-saml.

What do you think? 👍

alranel added a commit that referenced this issue Dec 7, 2018
@alranel
Copy link
Member

alranel commented Dec 7, 2018

Thank you for the report! Switched to text/template which does not provide XML escaping but it's fine for our needs.

The SAML package you linked actually looks abandoned, and generates invalid metadata: RobotsAndPencils/go-saml#9

@alranel alranel closed this as completed Dec 7, 2018
@alranel alranel added the bug Something isn't working label Dec 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants