-
Notifications
You must be signed in to change notification settings - Fork 501
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
OpenAPI definition endpoint #10328
Merged
Merged
OpenAPI definition endpoint #10328
Changes from 40 commits
Commits
Show all changes
51 commits
Select commit
Hold shift + click to select a range
e64bc66
Plugin initial config
jp-tosca d717ad1
Initial changes to provide OpenAPI definition
jp-tosca 63df178
Added integration test
jp-tosca 77f9c61
Imports fix
jp-tosca 2da5b05
Add patchnotes
jp-tosca 9f528d8
Update the changelog
jp-tosca a8c4c06
Update src/main/java/edu/harvard/iq/dataverse/api/Info.java
jp-tosca a859a50
Update doc/release-notes/10236-openapi-definition-endpoint.md
jp-tosca 8d41917
Update doc/release-notes/10236-openapi-definition-endpoint.md
jp-tosca 9405d54
Add native API docs
jp-tosca 468672e
Remove generated definitions
jp-tosca 3a0daa6
Add to gitignore generated openapi files
jp-tosca 3a699a8
Updates to docs
jp-tosca 4b6a9fd
Ignore files correction
jp-tosca c5bfc83
Branch update
jp-tosca 3f3145e
Remove files created by the plugin
jp-tosca 094105d
Changes to move the definition files to META-INF
jp-tosca 9d563eb
Changes to move the definitions to WEB-INF
jp-tosca f2f815c
Changes to get the files from META-INF
jp-tosca 058a4e5
Changed the phase of execution of the smallrye plugin
jp-tosca df9a680
Changes of names to improve the generation of the spec
jp-tosca 2873d07
Add support for OpenAPI annotations and documents the version endpoint
jp-tosca 25bebdc
Merge remote-tracking branch 'origin/develop' into openapi-definition…
jp-tosca fdb85c6
Multipart Annotations
jp-tosca e5c60f3
Merge remote-tracking branch 'origin/develop' into openapi-definition…
jp-tosca 80ebbf7
Typos correction
jp-tosca e1e19d7
Merge remote-tracking branch 'origin/develop' into openapi-definition…
jp-tosca b181899
Changes for tags
jp-tosca f8fa32b
Renaming of methods
jp-tosca 7d45759
Changes to the endpoint
jp-tosca c009cc7
Added test
jp-tosca 779897e
Add test
jp-tosca a0141cd
Deleted extra import
jp-tosca 9cdd9dc
Merge branch 'develop' into openapi-definition-endpoint
jp-tosca 9ca4111
Merge remote-tracking branch 'origin/develop' into openapi-definition…
jp-tosca b7ba865
Merge remote-tracking branch 'origin/develop' into openapi-definition…
jp-tosca 59fc8ce
Docs updated
jp-tosca 5ad07f7
openapi doc tweaks #9981 #10236
pdurbin b99b194
improve release note #9981 #10236
pdurbin f265744
Remove old test and changes response to JSON
jp-tosca 4ea444a
stub out guidance on openapi validation #9981 #10236
pdurbin 6fe471f
add InfoIT to list of tests
pdurbin 31a65a7
use description of Dataverse from website
pdurbin 5cbad6d
mention status codes in openapi doc
pdurbin 0d656a1
update api faq about changelog, link to breaking changes doc
pdurbin eb9cf7c
typo
pdurbin 678865d
Change to OpenApi
jp-tosca d23d78c
Changes to docs
jp-tosca 942d473
Name fix
jp-tosca 683ea85
Removing the multipart from unirest
jp-tosca ef267d0
Merge remote-tracking branch 'origin/develop' into openapi-definition…
jp-tosca File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,8 @@ | ||
In Dataverse 6.0 Payara was updated, which caused the url `/openapi` to stop working: | ||
|
||
- https://github.com/IQSS/dataverse/issues/9981 | ||
- https://github.com/payara/Payara/issues/6369 | ||
|
||
When it worked in Dataverse 5.x, the `/openapi` output was generated automatically by Payara, but in this release we have switched to OpenAPI output produced by the [SmallRye OpenAPI plugin](https://github.com/smallrye/smallrye-open-api/tree/main/tools/maven-plugin). This gives us finer control over the output. | ||
|
||
For more information, see the section on [OpenAPI](https://dataverse-guide--10328.org.readthedocs.build/en/10328/api/getting-started.html#openapi) in the API Guide. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm the one who added Unirest but it's one of the libraries I'm hoping we can remove some day in favor of using the Jakarta EE standard. Do we need it here? I see a possible solution at https://stackoverflow.com/questions/64079847/how-to-provide-swagger-annotation-for-multipartformdatainput-in-resteasy-with-qu/65339649#65339649 . Is it worth trying it here and other places in this PR where Unirest is being used?