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

Error when creating a SAML group with / in the name #172

Open
bedinger opened this issue Jul 27, 2023 · 0 comments
Open

Error when creating a SAML group with / in the name #172

bedinger opened this issue Jul 27, 2023 · 0 comments

Comments

@bedinger
Copy link
Contributor

Similar to #26, when creating a SAML group with a / in the name, an error is returned, because the name parameter is not URI-encoded by the provider.

Example

resource "splunk_admin_saml_groups" "demo_users" {
  name          = "demo/users"
  roles         = ["user"]
}

Output (abbreviated)

$ terraform apply

Terraform used the selected providers to generate the following execution plan. Resource actions are indicated with the following symbols:
  + create
  ~ update in-place

Terraform will perform the following actions:

  # splunk_admin_saml_groups.demo_users will be created
  + resource "splunk_admin_saml_groups" "demo_users" {
      + id    = (known after apply)
      + name  = "demo/users"
      + roles = [
          + "user",
        ]
    }

Plan: 1 to add, 0 to change, 0 to destroy.

Do you want to perform these actions?
  Terraform will perform the actions described above.
  Only 'yes' will be accepted to approve.

  Enter a value: yes

splunk_admin_saml_groups.demo_users: Creating...

│ Error: Invalid custom action for this internal handler (handler: SAML-groups, custom action: users, eai action: list).
│
│   with splunk_admin_saml_groups.demo_users,
│   on main.tf line 35, in resource "splunk_admin_saml_groups" "demo_users":
│   35: resource "splunk_admin_saml_groups" "demo_users" {

The fix for this will likely be similar to the one introduced in #29. I will be glad to submit a PR to address this.

bedinger pushed a commit to bedinger/terraform-provider-splunk that referenced this issue Jul 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant