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

Design Issue: Including Directories Outside of WLP Root #30320

Open
rsherget opened this issue Dec 2, 2024 · 2 comments
Open

Design Issue: Including Directories Outside of WLP Root #30320

rsherget opened this issue Dec 2, 2024 · 2 comments

Comments

@rsherget
Copy link
Contributor

rsherget commented Dec 2, 2024

Background

Users can use the <include> tag to include extra configuration files to the server. They can do this by adding individual files or by adding an entire directory that will add all the config files within that directory.

Problem

We currently do not add any configuration files through directories that are outside of the wlp directory or whatever directory is specified by java.io.tmpdir which by default is just the /tmp directory. However, users can specify individual config files outside of the /wlp and /tmp directories.

Questions

  • Do we not go through files within a directory outside of /wlp or /tmp for security reasons?

Proposed Solution

  1. Make it so that config files within directories can be specified even if the directory is outside the /wlp or /tmp directories.
  2. Add to docs to explain the difference in behavior between including config files by specifying directories versus specifying individual config files.
@rsherget rsherget moved this to Backlog (add new here) in Design Issues Dec 2, 2024
@rsherget
Copy link
Contributor Author

rsherget commented Dec 3, 2024

@cbridgha cbridgha moved this from Backlog (add new here) to Needs Implemention in Design Issues Dec 17, 2024
@cbridgha
Copy link
Member

Ricky for next year, I think we were going to ask a couple other people on some history here, also look at our "packaging" behavior and see if also filters, or doesn't allow these directories?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Needs Implemention
Development

No branches or pull requests

2 participants