Duplicated headers in request body in generated docs (POST, PATCH, etc.) about express-zod-api HOT 10 CLOSED

I assume this commit is what illustrates the issue

ccc098e

from express-zod-api.

In the screenshots, you can see that I have specified the x-request-id header as input for both the GET and POST endpoint, however, one shows up correctly as a header in the swagger doc and one only as the part of request body.

I have tried setting the inputSources to explicitly include the header field and also using just the defaults, both result in the same generated doc.

Does this example make the problem clear?

from express-zod-api.

I see, it's duplicated, @boarush
Present in both "parameters" and the "body"

from express-zod-api.

I think that when getting the requestBody here, it needs to exclude the headers if they're enabled.

from express-zod-api.

🚀 Should be fixed in v19.1.1, @boarush
In case it's not, please open another issue.
Thank you once again for your contribution!

from express-zod-api.

@boarush , this line controls the appearance of the parameter in headers when you generate the documentation:

Which means, as described in README, you should:

1. Enable headers in inputSources;
2. The header name should starts with x-.

In your commit first step is missing and that's why it's treated differently.
The step is missing, because you made changes to the example that is not configured to handle headers.

though after specifying the inputSources in the config, request works in both cases

So... What's the issue with setting inputSources?
Headers in input is the opt-in feature that you can enable that way.
Please clarify.

Thank you for the wonderful work! Really simple working with the library!

Thank you as well, appreciate it :)

from express-zod-api.

Ok. I need to make a test...

from express-zod-api.

On your screenshots both actually appear wrong, @boarush .
Even for GET request it says that the placement is query, but it should be header

And that's very confusing, because for that case I already have a test.

from express-zod-api.

I think I mixed up the screenshots, those were for when I was not including the inputSources. Following are for when I am including the inputSources:

GET is absolutely correct, POST is showing double fields.

My bad, sorry for the confusion.

from express-zod-api.

Confirmed, @boarush

Thank you so much for informing me and for the detailed explanation.

from express-zod-api.