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

[Feature Request] Directory Listing Control and Initial Directory Configuration for S3 Buckets" #744

Open
mauroalexandre opened this issue Sep 18, 2024 · 3 comments

Comments

@mauroalexandre
Copy link

Hello, congratulations on the fantastic project.

I would like to ask a question. Unfortunately, S3 does not allow through IAM to prevent a user from seeing the list of directories in an S3 Bucket. Therefore, when the user logs in, they land on a directory listing they may not always have access to. I have two questions:

1 - Is it possible to address this situation through configurations?

2 - If not, may I implement and submit a Pull Request to enable configuring a 'start directory' so that the user is directed to a specific directory after login? Would this feature be adherent to the project?

Thank you.

@mickael-kerjean
Copy link
Owner

Yes, this is what the "path" field is about:

screenshot-20240919-010904Z-all

If you put: /bucket/ it will chroot the user within a bucket and won't be running any bucketlisting operation as it only happen at the root

@mauroalexandre
Copy link
Author

@mickael-kerjean

Cool! But would it be possible to configure this at the connection level? So that this parameterization is not user-dependent?

@mickael-kerjean
Copy link
Owner

mickael-kerjean commented Sep 19, 2024

yes, this is precisely what the auth middleware from the admin console is for. You can build those path dynamically based on whatever attribute is being passed through or even build rules around that as well. I've revamped the doc in there https://www.filestash.app/docs/install-and-upgrade/#passthrough-pattern and https://www.filestash.app/docs/install-and-upgrade/#facade-pattern 2 days ago to make those sort of things clearer but not sure if that's really good yet. Happy to get feedbacks as if it's not 100% clear, it means it got to be improved a lot more

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

No branches or pull requests

2 participants