Skip to content

Fix "Controller" suffix removal#5312

Open
marius-hofmann wants to merge 1 commit into
RicoSuter:masterfrom
marius-hofmann:fix/controller-suffix-removal
Open

Fix "Controller" suffix removal#5312
marius-hofmann wants to merge 1 commit into
RicoSuter:masterfrom
marius-hofmann:fix/controller-suffix-removal

Conversation

@marius-hofmann
Copy link
Copy Markdown

* Only remove last occurrence of "Controller" when processing controller name in `WebApiOpenApiDocumentGenerator`
@marius-hofmann
Copy link
Copy Markdown
Author

@lahma thanks for having a look at my PR. I'm sorry, that the tests failed, but I think, the failing is not caused by my changes. It seems like http://petstore.swagger.io/v2/swagger.json returned a 502 during the test execution. I don't know if this is a permanent issue or just a short service outage. Could you re-run the checks in order to verify if there is a more permanent problem?

@RicoSuter
Copy link
Copy Markdown
Owner

@marius-hofmann should we do the same for the asp.net core implementation as well?

@marius-hofmann
Copy link
Copy Markdown
Author

@marius-hofmann should we do the same for the asp.net core implementation as well?

@RicoSuter What exactly do you mean? I could not find another position which uses this logic.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants