Jan Meiswinkel
Jan Meiswinkel
Good point, thanks. About the other point: Do you mean unsandboxed pipelines that run potentially dangerous functions that have been put on the allowlist?
> > Do you mean unsandboxed pipelines that run potentially dangerous functions that have been put on the allowlist? > > No, users can choose to disable the sandbox for...
> Our general policy is to review proposed API additions along with the proposed consumers. This can help us better understand the intended use case in order to properly review...
Hey @basil, Please let me know if this PR needs further justification. Thanks!
Thanks for your review, @res0nance. Very good point! My idea was to get the user in the listener implementation. It looked like the User object was still accessible there but...
Hi, I will be very busy for the next two weeks and probably not able to work on this PR during that time. Please feel free to continue reviewing but...
Hey @res0nance, sorry for the delay. I implemented the requested change. Thanks for the hint! PR is ready for review again.
@mawinter69 @basil Do you want to review this PR or should I tag the reviewer team?
Is this PR ready for merge? :) If anything else should be done on my side, please let me know.
> Please add a changelog entry Done