-
Notifications
You must be signed in to change notification settings - Fork 315
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
feat: add saml client installation provider #263
feat: add saml client installation provider #263
Conversation
Hey @Filirom1, thanks for the PR! I think the code you've submitted looks good so far. I really like the example you posted in this PR, that's a good use case for this data source. If you'd like, you could include that example in the docs to show off a real use case for this. I've never personally used the "Installation" tab for a SAML client before, but based on the code you've submitted, it looks like you're expecting an XML document to be returned in the You can look at examples for parsing xml in the If you have any specific questions about any of this please feel free to let me know. Thanks! |
… of https://github.com/Filirom1/terraform-provider-keycloak into data_source_keycloak_saml_client_installation_provider
Is it OK for you @mrparkers ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks for the PR!
so for setting up a SAML client for AWS...
what are these files? Where do I get them? |
Hello,
I need to retrieve the SAML installation XML to inject it in AWS SAML Provider
I started this Pull Request that is working, but I will need help if you need tests or refactoring because I am not fluent in golang yet :-)
Cheers
Romain