[
https://issues.apache.org/jira/browse/BOOKKEEPER-959?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15609938#comment-15609938
]
ASF GitHub Bot commented on BOOKKEEPER-959:
-------------------------------------------
Github user merlimat commented on the issue:
https://github.com/apache/bookkeeper/pull/67
The change looks OK to me. It's much easier to get around compared to the
Protobuf extension. I guess we (Yahoo) are the only current users of this
framework but I prefer to break the compatibility right now and have a better
solution.
The only point I'd like to make sure is how to transition a running cluster
from no-auth to auth enabled and back. Eg. if a client is sending auth
commands, will the bookie reply OK if auth is disabled?
> ClientAuthProvider and BookieAuthProvider Public API used Protobuf Shaded
> classes
> ---------------------------------------------------------------------------------
>
> Key: BOOKKEEPER-959
> URL: https://issues.apache.org/jira/browse/BOOKKEEPER-959
> Project: Bookkeeper
> Issue Type: Bug
> Components: bookkeeper-client, bookkeeper-server
> Affects Versions: 4.4.0
> Reporter: Enrico Olivelli
> Assignee: Enrico Olivelli
> Priority: Blocker
> Fix For: 4.5.0
>
>
> With 4.4.0 we introduced the ability to implement custom authentication
> plugins.
> The new interfaces ClientAuthProvider and BookieAuthProvider depend on
> ExtensionRegistry, which is a shaded dependency.
> As a consequence it is not possibile to implement any custom auth provider in
> code outside the project, because shaded/relocated dependencies cannot be
> used.
> We need to break the actual interface and introduce a new way to implement
> such plugins in a portable way
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
