Sounds Good. I will update the KIP to include an extension that authenticates Basic Auth Credentials against a properties file. Let me know if that works.
Thanks Magesh On Wed, Apr 11, 2018 at 9:51 AM, Gwen Shapira <g...@confluent.io> wrote: > Not a blocker, but it will continue a good tradition to add something > simple that works. I'm thinking plain-text auth and maybe ZK storage (like > the Kafka authorizer). It doesn't have to provide real security in any > sense, but it will allow people to test and experiment. Otherwise, if I > write an implementation and it doesn't work, it will be harder to know if > it is me or Kafka... > > On Wed, Apr 11, 2018 at 9:47 AM, Magesh Nandakumar <mage...@confluent.io> > wrote: > > > Hi Gwen, > > > > Thanks for your feedback. As part of this KIP, I was planning on adding a > > skeleton implementation in the examples but that can be used only as a > > reference. At this moment, there is no plan to add a concrete > > implementation of the plugin. Let me know your thoughts :). > > > > Thanks, > > Magesh > > > > On Wed, Apr 11, 2018 at 9:42 AM, Gwen Shapira <g...@confluent.io> wrote: > > > > > Thanks Magesh! The lack of authentication and authorization in the REST > > API > > > is definitely a painpoint and something that can prevent a company from > > > adopting connect (or surviving an audit after they did!). > > > > > > One thing that isn't clear to me from the KIP: In the past when we > > > contributed pluggable interfaces, we always contributed a simple > > > implementation that can be used for testing, experimentation and as an > > > example to future implementers. The JSON converter is one such example. > > Or > > > the ZK authorizer. > > > > > > Do you plan on including a simple implementation as part of this KIP as > > > well? > > > > > > Gwen > > > > > > On Tue, Apr 10, 2018 at 10:14 PM, Magesh Nandakumar < > > mage...@confluent.io> > > > wrote: > > > > > > > Hi, > > > > > > > > We have posted KIP-285: Connect Rest Extension Plugin to add the > > ability > > > to > > > > provide Rest Extensions to Connect Rest API. > > > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP- > > > > 285%3A+Connect+Rest+Extension+Plugin > > > > > > > > Please take a look. Your feedback is appreciated. > > > > > > > > Thanks, > > > > > > > > Magesh > > > > > > > > > > > > > > > > -- > > > *Gwen Shapira* > > > Product Manager | Confluent > > > 650.450.2760 | @gwenshap > > > Follow us: Twitter <https://twitter.com/ConfluentInc> | blog > > > <http://www.confluent.io/blog> > > > > > > > > > -- > *Gwen Shapira* > Product Manager | Confluent > 650.450.2760 | @gwenshap > Follow us: Twitter <https://twitter.com/ConfluentInc> | blog > <http://www.confluent.io/blog> >
