-
Notifications
You must be signed in to change notification settings - Fork 186
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
Move security api from plugins to clients #442
Conversation
Signed-off-by: florian <florian@harfanglab.fr>
Signed-off-by: florian <florian@harfanglab.fr>
Codecov Report
@@ Coverage Diff @@
## main #442 +/- ##
=======================================
Coverage 71.49% 71.49%
=======================================
Files 81 81
Lines 7668 7668
=======================================
Hits 5482 5482
Misses 2186 2186
|
How does this impact end-users? Anything I'd have to change in my code when upgrading? |
@dblock This not impact end-users, security API is still accessible from the client, I haven't changed the tests, it still works as before. |
That's a breaking change :( Is there a way to proxy the old to the new so we don't have to bump the major version? |
@dblock This change is not yet released. |
Oh! Good. |
Description
I move the
SecurityClient
from the plugins directory to the clients directory, accordingly to the @saimedhi comment (#399 (comment)).