Uploaded image for project: 'Camel'
  1. Camel
  2. CAMEL-17717

REST DSL securityRequirements cleanup

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • 3.16.0
    • camel-core
    • None
    • Unknown

    Description

      RestDefinition (rest.json) has securityRequirements property of type SecurityRequirementsDefinition

      {
        "model": {
          "kind": "model",
          "name": "rest",
          "title": "Rest",
          "javaType": "org.apache.camel.model.rest.RestDefinition",
      ...
        },
        "properties": {
      ...
          "securityRequirements": { "kind": "element", "displayName": "Security Requirements", "label": "security", "required": false, "type": "object", "javaType": "org.apache.camel.model.rest.SecurityRequirementsDefinition", "deprecated": false, "autowired": false, "secret": false, "description": "Sets the security requirement(s) for all endpoints." },
      ...
      }
      

      However SecurityRequirementsDefinition has only one property: securityRequirements

      {
        "model": {
          "kind": "model",
          "name": "securityRequirements",
          "javaType": "org.apache.camel.model.rest.SecurityRequirementsDefinition",
      ...
        },
        "properties": {
          "securityRequirements": { "kind": "element", "displayName": "Security Requirements", "required": true, "type": "array", "javaType": "java.util.List<org.apache.camel.model.rest.SecurityDefinition>", "oneOf": [ "securityRequirement" ], "deprecated": false, "autowired": false, "secret": false, "description": "Security requirement configurations" }
        }
      }
      

      Could we remove intermediate SecurityRequirementsDefinition and have securityRequirements property directly in RestDefinition?

      Attachments

        Issue Links

          Activity

            People

              essobedo Nicolas Filotto
              marat.gubaidullin@gmail.com Marat Gubaidullin
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: