Required property should not be flagged as invalid when using inside allOf

Issue inspector finds that a property inside required don’t exists, but it’s false. In particular: Schema lists property "from" as required, but it does not exist. No additional information found.

In fact it exists in another schema merged using allOf:

components:
    schemas:
        Username:
            minLength: 5
            type: string
        NewTransaction:
            allOf:
                -
                    required:
                        - from
                        - to
                        - description
                        - value
                    type: object
                -
                    $ref: '#/components/schemas/UpdateTransaction'
        UpdateTransaction:
            type: object
            properties:
                from:
                    $ref: '#/components/schemas/Username'
                to:
                    $ref: '#/components/schemas/Username'
                value:
                    type: number
                description:
                    minLength: 1
                    type: string
        Transaction:
            allOf:
                - $ref: '#/components/schemas/UpdateTransaction'
                -
                    required:
                        - from
                        - to
                        - id
                        - description
                        - value
                        - at
                    type: object
                    properties:
                        id:
                            type: string
                        at:
                            format: date-time
                            type: string

[EricWittmann]

Ah ha - well then I think that is the issue: you can require a property without defining it.

So the validation rule I added for this was simply erroneous. I think what I’ll do is disable it (for now) until I have a way of enabling/disabling rules in the Apicurio UI. Once that’s done, we can start doing all sorts of interesting things with additional validation rules like this.

[EricWittmann]

Closing this since I have disabled the offending validation rule.