-
Notifications
You must be signed in to change notification settings - Fork 269
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
V2 api auth support #1172
Merged
EricWittmann
merged 3 commits into
Apicurio:feat/artifact-groups
from
carlesarnal:v2-api-auth-support
Jan 29, 2021
Merged
V2 api auth support #1172
EricWittmann
merged 3 commits into
Apicurio:feat/artifact-groups
from
carlesarnal:v2-api-auth-support
Jan 29, 2021
Conversation
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
EricWittmann
approved these changes
Jan 29, 2021
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.
Looks good, thanks. Does the Auth test work for you now on this branch?
It works for the v1 but (obviously) not for the v2. |
EricWittmann
added a commit
that referenced
this pull request
Mar 1, 2021
* JAX-RS generated classes for v2 of the Registry API. * Lots of refactoring to start implementing v2 of the rest api * JAX-RS generated classes for v2 of the Registry API. * Added groupId concept to the storage layer * JAX-RS generated classes for v2 of the Registry API. * Update ArtifactgroupsResource.java * Update ArtifactMetaData.java * Update EditableMetaData.java * Update SearchedVersion.java * Update VersionMetaData.java * Updated the REST API with some recent changes * Handle serializer cache, simplify Streams storage (#1163) * GlobalId Strategy should seed the Schema cache if it registers a schema #1148 * Simplify Streams storage, no need for async global id mapping. * Updated the asyncmem storage * JAX-RS generated classes for v2 of the Registry API. * Regenerated the jax-rs layer after modifications to the v2 api design * JAX-RS generated classes for v2 of the Registry API. * Additional changes to the v2 REST api * added jax-rs test for the /groups v2 api endpoint * V2 api auth support (#1172) * Add security paths for v2 api endpoints * Remove localhost placeholder * Simplify permission paths * Fixed a failing unit test * Added a test for multiple groups. * JAX-RS generated classes for v2 of the Registry API. * Added more tests and updated the in-memory storage to support contentId and contentHash * JAX-RS generated classes for v2 of the Registry API. * Implemented the /ids API endpoints and created tests for them * minor change to the auth test and its superclass * updated the asyncmem and ispn storages with latest changes * removed an unused interface in the UI service * Created a search REST test * Stop polluting compatibility layer with REST beans * Added UI support for groups in the Artifacts page and Upload Artifact modal * Added UI support for groups. * Rest client using java 11 http client. (#1195) * Plain java rest client initial implementation * Extract client creation into a factory * Add json body handler * Use json body handler * Use entry set * Extract methods to request handler * SQL storage - artifact groups (#1189) * sql storage - artifact groups * undo unneeded test changes * searchArtifacts by groupId - make exact search * add groupId test * minor code readability changes in the new rest client impl * Implemented the /ids operations in the new rest client * Fix testCorrectGroup test * fixed a test and disabled the auth test when running on windows * Search artifacts and simplified request handler (#1202) * Fix accept and content type headers * Change artifact name * Artifact search * Improve rest client code format * Simplify request handler * Add search versions * Fix list query params * Updated the maven plugins to use the v2 api. Also updated how the mojos are configured * Java 11 rest client 1173 (#1204) * Fix format issues * Add admin resources to rest client (#1206) * Use io utils when dealing with streams * Add artifact rule tests * Add admin resources to rest client * Implement admin resources * Add test comments * Add auth capabilities * Refactor request handler and add initial auth capabilities * Fix empty response expected issue * Add request headers customization capabilities (#1210) * Add request headers customization capability * Improve json error handling (#1211) * v2 api - dynamic log configuration (#1190) * api v2 dynamic log configuration impl * dynamic log configuration: sql storage * Simplify java 11 rest client (#1212) * Fix log operations path and enable tests (#1213) * Fixed a unit test and removed an unused import * fixed the test URL for two mojo tests * New integration-tests + serdes 2.0 (#1218) * serdes refactoring - serdes 2.0 * serde 2.0 - improvements * improve serdes and add serdes QuarkusTests * WIP: integration-tests 2.0 * serdes-2.0 impl improvements * new client bug fixes * new integration-tests * test fixes * fix tests deps in storages * fix sql storage bug * fix loggers tests * fix listArtifactsInGroup offset and limit parameters * disable infinispan tests * build integration tests common * fix legacy tests deps issue * re-implement more integration-tests * fix TestUpdateRegistryMojoTest * Regenerate JAX-RS layer with addition of quarkus annotation on beans (#1223) * removed some usages of "var" instead of specific var types * JAX-RS generated classes for v2 of the Registry API. * Update RuleViolationError.java * CLI - use v2 api client, use quarkus command mode and compile the tool to native (#1219) * Add tls and custom configuration support (#1222) * Add tls and custom configuration support * Fix compilation error in serdes * Fix bad param value * Feat/artifact groups test old serdes release (#1227) * revert #1132 in v1 API for compatibility with old client * legacy-tests test old client version(1.3.2.Final) with v1 api * fix testLoggersCRUD test (#1228) * JAX-RS generated classes for v2 of the Registry API. * Update RuleViolationError.java * Maven build cleanup, removed deprecated v1 client and serdes, disabled connector temporarily * refactor converters to use serdes 2.0 * Uncomment converter and converter distro * updated kafka+sql storage to support groups and log configs * Fixed some bugs in the kafkasql groupId support. * Fix potential mbean registration error in kafkasql * Minor improvements found when updating apicurio-registry-examples * re-organized the API endpoints. Fixes #1238 * Created an API listing page located at /apis - now all supported APIs are documented there * Back to using the client to delete global rules for all tests. Set client to always use HTTP/1 * Adapt auth paths to new api structure (#1255) * Serdes improvements: config classes, protobuf textual schema support, headers handler, more tests, schema resolver api, deserializers, ... (#1256) * separated out schema resolver functionality to provide a nice base class for custom resolvers * Incorrect transitive compatibility checking for JSON schemas. Fixes #1239 (#1257) * compatibility request filter for the old /api/ path (#1258) * Support "default" groupId in v2 to simulate "no group" and provide compatibility with v1 api * Support for "default" groupId in v2 API (#1259) * Support "default" groupId in v2 to simulate "no group" and provide compatibility with v1 api * adapt tests to default groupId Co-authored-by: Fabian Martinez <[email protected]> * Add artifact groups streams storage 1180 (#1264) * Change artifact id by artifact key * Add artifact key support to streams storage layer * Add artifact key serdes * Partially fix artifacts search * Fix metadata search * Add fixme search metadata * Add groups and content segregation to streams storage * Fix update state method call * Fix content id not being returned in metadata * Enable streams storage variant tests * Wip ~ test cluster it with streams * Change client response from class to type reference * Remove additional version * Fix tabs usage * fix GH workflows * fix tests * remove cache dependencies action and run all tests on master branch * Fix invalid schema integration test for now * disable ISPN build * disable infinispan integration tests Co-authored-by: Aleš Justin <[email protected]> Co-authored-by: Carles Arnal <[email protected]> Co-authored-by: Fabian Martinez <[email protected]> Co-authored-by: Fabian Martinez <[email protected]> Co-authored-by: Jakub Senko <[email protected]> Co-authored-by: Carles Arnal <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Adds HTTP permission path for version 2 of the API.