Thanks Chris. I have added that as well.
I think I can start the vote on this KIP if there are no further comments.
Thanks.
Regards,
Tina
On Tue, Dec 12, 2023 at 1:43 PM Chris Egerton
wrote:
> Thanks Tina! LGTM as long as we take care to document that recursive access
> to directories will be
Thanks Tina! LGTM as long as we take care to document that recursive access
to directories will be granted when we release this feature.
On Tue, Dec 12, 2023 at 8:16 AM Gantigmaa Selenge
wrote:
> Hi Chris,
>
> Thank you for the feedback.
>
>
> 1. Addressed
>
>
> 2. I have updated the type to be
Hi Chris,
Thank you for the feedback.
1. Addressed
2. I have updated the type to be List. The configure() function is more
likely to process the value as String and convert to List using the comma
separation but I think it makes sense to specify it as List, as that is the
final type.
3. Yes
Hi Tina,
Thanks for the KIP! Looks good overall. A few minor thoughts:
1. We can remove the "This page is meant as a template for writing a KIP"
section from the beginning.
2. The type of the allowed.paths property is string in the KIP, but the
description mentions it'll contain multiple comma-s
Thank you Mickael.
I'm going to leave the discussion thread open for a couple more days and if
there are no further comments, I would like to start the vote for this KIP.
Thanks.
Regards,
Tina
On Wed, Dec 6, 2023 at 10:06 AM Mickael Maison
wrote:
> Hi,
>
> I'm not aware of any other mechanisms
Hi,
I'm not aware of any other mechanisms to explore the filesystem. If
you have ideas, please reach out to the security list.
Thanks,
Mickael
On Tue, Dec 5, 2023 at 1:05 PM Gantigmaa Selenge wrote:
>
> Hi everyone,
>
>
> Apologies for the very delayed response. Thank you both for the feedback.
Hi everyone,
Apologies for the very delayed response. Thank you both for the feedback.
> For clarity it might make sense to mention this feature will be useful
when using a ConfigProvider with Kafka Connect as providers are set in
the runtime and can then be used by connectors. This feature h
Hey Tina,
Thanks for the KIP! Unrestricted file system access over a REST API is
an unfortunate anti-pattern, so I'm glad that you're trying to change
it. I had a few questions, mostly from the Connect perspective.
1. In the past Connect removed the FileStream connectors in order to
prevent a RES
Hi Tina,
Thanks for the KIP.
For clarity it might make sense to mention this feature will be useful
when using a ConfigProvider with Kafka Connect as providers are set in
the runtime and can then be used by connectors. This feature has no
use when using a ConfigProvider in server.properties or in
Hi everyone,
Please let me know if you have any comments on the KIP.
I will leave it for a few more days. If there are still no comments, I will
start the vote on it.
Regards,
Tina
On Wed, Oct 25, 2023 at 8:31 AM Gantigmaa Selenge
wrote:
> Hi everyone,
>
> I would like to start a discussion o
10 matches
Mail list logo
