Skip to content

Conversation

@bjornblomqvist
Copy link

Hi, prior to version 1.6.5 I could you use both a spec file and generated spec from annotations. This behavior is something I need to be able to upgrade to a later version. I am not sure if this feature that I have been using was even ment to work but it did and I would very much appreciate if we could bring it back.

You can find in this pull request a test and a fix for my problem. I think I put the test in the wrong place so I would appreciate if you could help me improve the test. Also as I don't know the code I think the fix might also be in the wrong place.

Best regards
Björn Blomqvist

@bjornblomqvist bjornblomqvist changed the title Using both generated and configured specs stoped working in 1 Apr 21, 2023
@bjornblomqvist
Copy link
Author

bjornblomqvist commented May 2, 2023

@bnasslahsen what do you think about this pull request?

@bjornblomqvist bjornblomqvist force-pushed the using-both-generated-and-configured-specs-no-longer-works branch from 2214fc7 to de848d4 Compare May 4, 2023 07:22
@bjornblomqvist
Copy link
Author

The only change was to rebase on latest master

I don't fully understand the fix but forcing SpringBootTest runner to
not reuse the context solves the issue with SpringDocApp1451Test.java
always failing with the following.

"WebClientRequestException: Connection refused: localhost/127.0.0.1:9086"

BUGFIX
Being able to use both a spec file and generated spec stopped working
in version 1.6.5. The test shows the behavior found 1.6.4 and the fix
brings back that behavior.

BUGFIX
@bjornblomqvist bjornblomqvist force-pushed the using-both-generated-and-configured-specs-no-longer-works branch from de848d4 to 41a988e Compare May 4, 2023 08:12
@bjornblomqvist
Copy link
Author

Last force push was to remove my employers email address.

@bjornblomqvist
Copy link
Author

@bnasslahsen have you had time to look at this yet?

@bnasslahsen
Copy link
Collaborator

@bjornblomqvist,

Not yet.
But it will be my next PR in the project to review.

@bnasslahsen bnasslahsen merged commit 41a988e into springdoc:master Jun 2, 2023
@bnasslahsen
Copy link
Collaborator

Thank you @bjornblomqvist for your PR.
It's now merged and also committed to 2.x branch as well.

bnasslahsen added a commit that referenced this pull request Jun 2, 2023
mpleine pushed a commit to mpleine/springdoc-openapi that referenced this pull request May 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants