Skip to content

add content type for ADFS FederationMetadata.xml#4

Merged
grubernaut merged 1 commit intohashicorp:masterfrom
kgcurran:master
Aug 16, 2017
Merged

add content type for ADFS FederationMetadata.xml#4
grubernaut merged 1 commit intohashicorp:masterfrom
kgcurran:master

Conversation

@kgcurran
Copy link
Copy Markdown
Contributor

@kgcurran kgcurran commented Aug 15, 2017

I would like to be able to pull ADFS federation metadata xml file directly in terraform for creating an AWS IAM SAML Provider

but it fails with error

data.http.adfs: Content-Type is not a text type. Got: application/samlmetadata+xml

data "http" "adfs" {
  url = "https://adfs.example.com/FederationMetadata/2007-06/FederationMetadata.xml"
}

resource "aws_iam_saml_provider" "adfsprod" {
  name = "adfs.example.com"
  saml_metadata_document = "${data.http.adfs.body}"
}

@iceycake

Copy link
Copy Markdown
Contributor

@grubernaut grubernaut left a 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 contribution!

@grubernaut grubernaut merged commit 0ea45bc into hashicorp:master Aug 16, 2017
@github-actions
Copy link
Copy Markdown

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators May 29, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants