Skip to content

Update samba.md to allow writes #1338

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

Merged
merged 3 commits into from
Jun 8, 2021
Merged

Conversation

neilh10
Copy link
Contributor

@neilh10 neilh10 commented Nov 17, 2019

As written it didn't work for me on the latest rasberrian/RaspberryPi3. Without the user share chmod it forbids access, and then with the specified samba uses nobody

As written it didn't work for me on the latest rasberrian/RaspberryPi3.   Without the user share chmod it forbids access, and then with the specified samba uses nobody
@JamesH65 JamesH65 changed the title Update samba.md to allow wirtes Update samba.md to allow writes Dec 17, 2019
@neilh10
Copy link
Contributor Author

neilh10 commented Dec 17, 2019

Thanks for looking at it. I've just noticed this conversation.

@ghost
Copy link

ghost commented May 23, 2020

This PR makes the shared directory read/write by all users, including guest. I don't think this was the intention of the instructions as they were originally written. I read it as showing how to configure the share so that only the pi user can connect to it, however it was missing the step where you add the Samba password using smbpasswd. You should not need the chmod part if you do the smbpasswd correctly, then use the correct username and password to connect to the Samba share.

Also, you don't need both read only = no and writable = yes - they have the exact same meaning.

@aallan
Copy link
Contributor

aallan commented Jun 7, 2021

In light of #1911 and the lack of progress in a long while I'm marking this PR as 7 day closure.

@JamesH65 JamesH65 merged commit a8f6fa6 into raspberrypi:master Jun 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants