Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[secure-mode] Add properties allowlist validation #2129

Merged
merged 3 commits into from
Jun 28, 2024

Conversation

ethnzhng
Copy link
Contributor

No description provided.

@ethnzhng ethnzhng requested review from zachgk, frankfliu and a team as code owners June 28, 2024 19:55
@siddvenk
Copy link
Contributor

I'd prefer a denylist instead of an allowlist With this, new configs are not allowed by default (maybe that's the desired behavior), but I feel like there are going to be new configs added that should be allowed more often than new configs added that are disallowed.

@ethnzhng ethnzhng force-pushed the secure-mode-patch branch from bb041b9 to 4f53236 Compare June 28, 2024 20:29
@ethnzhng ethnzhng force-pushed the secure-mode-patch branch from b9dc22f to e1c9644 Compare June 28, 2024 20:58
@ethnzhng ethnzhng force-pushed the secure-mode-patch branch from e1c9644 to d019e01 Compare June 28, 2024 21:00
@frankfliu frankfliu merged commit 707a9ab into deepjavalibrary:master Jun 28, 2024
8 checks passed
frankfliu pushed a commit to frankfliu/djl-serving that referenced this pull request Jun 28, 2024
frankfliu added a commit that referenced this pull request Jun 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants