Skip to content

Conversation

@zachgk
Copy link
Contributor

@zachgk zachgk commented May 29, 2024

Adds a new file to introduce the security policy. The initial version is based off of torchserve: https://github.com/pytorch/serve/blob/master/SECURITY.md.

@zachgk zachgk requested review from a team and frankfliu as code owners May 29, 2024 23:15
The default DJL Serving configuration in the container, which is executed by the docker entrypoint, will expose both the inference and management APIs set to `http://0.0.0.0:8080`.
This is designed for internal isolated services or development work. For other use cases, provide alternative configurations to avoid exposing the management API.

3. Be sure to validate the authenticity of all model files and model artifacts being used with DJL Serving.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We might want to mention about

  1. requirements.txt
  2. options.trust_remote_code
  3. environment variable

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added mentions on those topics as well. Feel free to let me know or just change it if it doesn't match up with what you had in mind

Adds a new file to introduce the security policy. The initial version is based
off of torchserve: https://github.com/pytorch/serve/blob/master/SECURITY.md.
@zachgk zachgk merged commit 2bf31b6 into deepjavalibrary:master May 30, 2024
@zachgk zachgk deleted the secMd branch May 30, 2024 19:11
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.

2 participants