You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@zeppelin.apache.org by "pjfanning (via GitHub)" <gi...@apache.org> on 2025/07/28 09:39:58 UTC
[PR] remove 'exceptions' in security.md [zeppelin-site]
pjfanning opened a new pull request, #30:
URL: https://github.com/apache/zeppelin-site/pull/30
This softens the language about Docker and K8s deployments making a big difference to the security model.
Wdyt @jongyoul, @Reamer, @zjffdu, @raboof ?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@zeppelin.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] remove 'exceptions' in security.md [zeppelin-site]
Posted by "Reamer (via GitHub)" <gi...@apache.org>.
Reamer commented on PR #30:
URL: https://github.com/apache/zeppelin-site/pull/30#issuecomment-3180354687
> Is it ok to get this merged?
In my opinion, Yes
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@zeppelin.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] remove 'exceptions' in security.md [zeppelin-site]
Posted by "pjfanning (via GitHub)" <gi...@apache.org>.
pjfanning commented on PR #30:
URL: https://github.com/apache/zeppelin-site/pull/30#issuecomment-3178397048
Is it ok to get this merged?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@zeppelin.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] remove 'exceptions' in security.md [zeppelin-site]
Posted by "jongyoul (via GitHub)" <gi...@apache.org>.
jongyoul merged PR #30:
URL: https://github.com/apache/zeppelin-site/pull/30
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@zeppelin.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] remove 'exceptions' in security.md [zeppelin-site]
Posted by "pjfanning (via GitHub)" <gi...@apache.org>.
pjfanning commented on PR #30:
URL: https://github.com/apache/zeppelin-site/pull/30#issuecomment-3126558636
> I think the direction matches the consensus in https://lists.apache.org/thread/167846z7rrjdjwz9vcxoxpnzqx0pok45 .
>
> I think 'recommends' still leaves a bit too much to interpretation: do we promise isolation or don't we? Perhaps something like: "This isolation can provide an operational benefit on large deployments, but is not intended as a security boundary: access to your Zeppelin instances should be restricted regardless of how they are deployed."?
change made
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@zeppelin.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org