fix: appfwprofile_sqlinjection_binding

[janschumann]

Updating binding fails.

Consider this resource:

resource "citrixadc_appfwprofile_sqlinjection_binding" "this" {
  name                 = citrixadc_appfwprofile.this.name
  sqlinjection         = "field"
  formactionurl_sql    = "https://example"
  as_scan_location_sql = "FORMFIELD"
  isregex_sql          = "NOTREGEX"
  as_value_type_sql    = "Keyword"
  as_value_expr_sql    = "example1"
  isvalueregex_sql     = "NOTREGEX"
  state                = "ENABLED"
}

After changing a value like this:

resource "citrixadc_appfwprofile_sqlinjection_binding" "this" {
  name                 = citrixadc_appfwprofile.this.name
  sqlinjection         = "field"
  formactionurl_sql    = "https://example"
  as_scan_location_sql = "FORMFIELD"
  isregex_sql          = "NOTREGEX"
  as_value_type_sql    = "Keyword"
  as_value_expr_sql    = "example2"
  isvalueregex_sql     = "NOTREGEX"
  state                = "ENABLED"
}

The update fails with

Error: [INFO] delete failed: 599 Netscaler specific error ({ "errorcode": 3130, "message": "No such SQLInjection check", "severity": "ERROR" })

Here is the nitro log for this update:

Command "unbind appfw profile appfw-profile-tf-tfi-fbt-smaragd-tcm-webcheck-htmlwaf -SQLInjection sqlinjection "
https://sqlinjection"
-location FORMFIELD -RuleType ALLOW" - Status "ERROR: No such SQLInjection check"

The problem is that in the delete process specifying the sqlinjection field is not sufficient for identifying the rule.

This PR adds fields to be able to apply the changes in the example above.

This may not be sufficient for all scenarios and also applies for other resources like appfwprofile_crosssitescripting_binding. We are currently testing those resources and will add more fixes accordingly.