Carmine DiMascio

Results 102 comments of Carmine DiMascio

@dnalborczyk i added some new multi-spec [tests](https://github.com/cdimascio/express-openapi-validator/blob/master/test/multi.spec/multi.spec.spec.ts). the tests essentially replicate the [example 2 multi-spec](https://github.com/cdimascio/express-openapi-validator/tree/master/examples/2-standard-multiple-api-specs). the tests succeed. one key difference is that the tests do not use operation handlers....

@babaliaris thanks. You are correct. It's not handling the date probably. Here is the offending [line]( https://github.com/cdimascio/express-openapi-validator/blob/master/src/framework/base.serdes.ts#L16) Would u be willing to post a PR with the fix?

@kutenai, thanks for the issue. Could u provide a canonical example that demonstrates the issue. If I have a reproducible example, I can get it a fix out relatively quickly....

@kutenai no problems. once you post the example, I'll certainly have a look enjoy your vacation!

@tumkurrunner can you provide the example request also, i see that the `DecisionRequest` requires `actor`, but `actor` is not defined as a property anywhere. it would be good to solve...

@tumkurrunner i'm working on an initial version of discriminator support. it will start by supporting top level types. will expand it to this use case. #461

Cool. Sounds like a reasonable workaround. Feel free to submit a PR. Happy to review

@bluenote10 Using the first form e.g. `nullableObjectMaybeNullA`, `nullable` would be ignored given its a sibling to the `$ref`, hence that won't work. The second form is, well, complicated, see https://github.com/OAI/OpenAPI-Specification/issues/1368...

@boks1971, thanks for this issue. An example will be helpful. Feel free to create a repo that demonstrates the bug. I'll have a look.

@alexandre-okidoo please submit a PR. Thanks!