-
Notifications
You must be signed in to change notification settings - Fork 9
remove superfluous parameters at the mapping stage #327
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
To make sure I understand your use case: For example: you have described a header in the OpenAPI that will be handled by a filter. Therefore it would not be needed in the endpoint interface method? |
My use case is currently only for the client. On the server, I agree it could be handled by a filter, or perhaps could be bundled into some other added parameter. |
hmm, I'm confused by client side. openapi-processor does generate server side code. How do you use it client side? Feign? 🤔 |
I just wrote an implementaiton of the generated interface (using REST-assured). |
I'll add it. |
available with 2025.2 |
The
mapping.yaml
file currently allows adding extra parameters, e.g. aHttpServletRequest
..For the client side in particukar, tt would also be usefult to generate interfaces in which named parameters coudl be dropped. These might represent common settings that are instead set by confiiguration or during initialisation, rathee than as part of each call, or just optional parameters the client does not send.
For example, something like:
The text was updated successfully, but these errors were encountered: