Validating required fields when using allOf

[nussbam]

When using allOf it is not checked if the fields referenced in required are existing

Example:

BaseData:
   properties:
      BaseProperty:
         type: string

ActualData:
    allOf:
      - $ref: '#/BaseData'
       - type: object
         required:
            - objectDoesntExist

This is a valid configuration according to the validator.

Expected behavior:

This should throw a validation error, hinting that the required field doesn't exist

[jdegre]

I think that the definition above is actually correct. It requires the type ActualData to comply with the following constraints:

• It must be an object
• It may have a property called BaseProperty and, if so, such property must be a string
• It must have a property called objectDoesntExist, but nothing is said about its type

It is true that in most cases a mismatch between 'properties' and 'required' is due to an error, but not necessarily.

[nussbam]

It must have a property called objectDoesntExist, but nothing is said about its type

It is not checked if it has a property called "objectDoesntExist".
The example is currently seen as valid, even though there is no property "objectDoesntExist", which I see as a misleading / wrong behavior