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

Request for samba share, restrict certain folders #3775

Open
Steven-park opened this issue Sep 26, 2024 · 3 comments · May be fixed by #3701
Open

Request for samba share, restrict certain folders #3775

Steven-park opened this issue Sep 26, 2024 · 3 comments · May be fixed by #3701

Comments

@Steven-park
Copy link

Describe the issue you are experiencing

I am unable to hide/restrict certain folders as I wish to expose the media folder only for media straming, but would be safer to restrict access to the other folders for Users.
This could help with a bit of unwanted or accidental tampering of the other files such as those in the config folder

What type of installation are you running?

Home Assistant OS

Which operating system are you running on?

Home Assistant Operating System

Which add-on are you reporting an issue with?

Samba share

What is the version of the add-on?

12.3.2

Steps to reproduce the issue

  1. Run samba share
  2. SMB to the exposed folders
  3. See other folders and accidently delete a backup
    ...

System Health information

System Information

version core-2024.9.3
installation_type Home Assistant OS
dev false
hassio true
docker true
user root
virtualenv false
python_version 3.12.4
os_name Linux
os_version 6.6.46-haos
arch x86_64
timezone Pacific/
config_dir /config
Home Assistant Community Store
GitHub API ok
GitHub Content ok
GitHub Web ok
HACS Data ok
GitHub API Calls Remaining 4998
Installed Version 2.0.1
Stage running
Available Repositories 1419
Downloaded Repositories 3
Home Assistant Cloud
logged_in true
subscription_expiration October 1, 2024 at 1:00 PM
relayer_connected true
relayer_region ap-southeast-1
remote_enabled true
remote_connected true
alexa_enabled false
google_enabled true
remote_server ap-southeast-1-2.ui.nabu.casa
certificate_status ready
instance_id f010ba8b354740ccb7545312372f6eea
can_reach_cert_server ok
can_reach_cloud_auth ok
can_reach_cloud ok
Home Assistant Supervisor
host_os Home Assistant OS 13.1
update_channel stable
supervisor_version supervisor-2024.09.1
agent_version 1.6.0
docker_version 26.1.4
disk_total 228.5 GB
disk_used 31.2 GB
healthy true
supported true
host_connectivity true
supervisor_connectivity true
ntp_synchronized true
virtualization
board generic-x86-64
supervisor_api ok
version_api ok
installed_addons Plex Media Server (3.6.0), Tailscale (0.22.0), Terminal & SSH (9.15.0), openWakeWord (1.10.0), Samba share (12.3.2), FTP (5.1.0), RPC Shutdown (2.5), File editor (5.8.0)
Dashboards
dashboards 3
resources 1
views 13
mode storage
Recorder
oldest_recorder_run September 17, 2024 at 4:49 AM
current_recorder_run September 25, 2024 at 1:02 AM
estimated_db_size 65.34 MiB
database_engine sqlite
database_version 3.45.3

Anything in the Supervisor logs that might be useful for us?

No response

Anything in the add-on logs that might be useful for us?

No response

Additional information

No response

@as-kholin
Copy link
Contributor

This is a not-rare request that is being worked on. See PR #3701 for current status and for any feedback.

Copy link

github-actions bot commented Nov 3, 2024

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale label Nov 3, 2024
@as-kholin
Copy link
Contributor

PR #3701 is working on this, and pending review

@github-actions github-actions bot removed the stale label Nov 3, 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 a pull request may close this issue.

2 participants