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

Application gateway: disabled rule groups #3356

Closed
wants to merge 4 commits into from

Conversation

timja
Copy link
Contributor

@timja timja commented May 1, 2019

Currently WIP

Having issues with the flatten, although the creation works fine

--- FAIL: TestAccAzureRMApplicationGateway_webApplicationFirewallDisabledRuleGroups (264.94s)
    testing.go:568: Step 0 error: errors during apply:

        Error: Error setting `waf_configuration`: waf_configuration.0.disabled_rule_group.0: '' expected a map, got 'slice'

          on /var/folders/6r/vwg96jz13pn1dlb6tk2_x6rm0000gn/T/tf-test386488536/main.tf line 48:
          (source code not available)


    testing.go:629: Error destroying resource! WARNING: Dangling resources
        may exist. The full state and error is shown below.

        Error: errors during refresh: Error setting `waf_configuration`: waf_configuration.0.disabled_rule_group.0: '' expected a map, got 'slice'

        State: <nil>
FAIL

Opening for comment on the design and any general feedback

  waf_configuration {
    enabled          = true
    firewall_mode    = "Detection"
    rule_set_type    = "OWASP"
    rule_set_version = "3.0"
    file_upload_limit_mb = 100
    request_body_check = true
    max_request_body_size_kb = 100

	disabled_rule_group {
		rule_group_name = "REQUEST-913-SCANNER-DETECTION"
		rules = [ 913100 ]
    }

	disabled_rule_group {
		rule_group_name = "REQUEST-920-PROTOCOL-ENFORCEMENT"
    }
  }

@ghost ghost added the size/L label May 1, 2019
@timja timja changed the title Application gateway: disabled rule groups WIP: Application gateway: disabled rule groups May 1, 2019
@ghost ghost added the documentation label May 2, 2019
@timja timja changed the title WIP: Application gateway: disabled rule groups Application gateway: disabled rule groups May 3, 2019
@bs-matil
Copy link
Contributor

bs-matil commented May 7, 2019

@timja would #3394 do the trick for you?

@timja
Copy link
Contributor Author

timja commented May 7, 2019

@bs-matil Yeah, there's one difference in that my PR allows optional rules so that the whole group is disabled.

I would like that supported as that matches the API docs

@timja
Copy link
Contributor Author

timja commented May 7, 2019

Closing in favour of: #3394

@timja timja closed this May 7, 2019
@timja timja deleted the feature/ag-enhanced-waf-config branch May 7, 2019 14:05
@ghost
Copy link

ghost commented Jun 7, 2019

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 [email protected]. Thanks!

@ghost ghost locked and limited conversation to collaborators Jun 7, 2019
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.

3 participants