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: read/write from/to samba (smb) drives #2042

Closed
dtenenba opened this issue Feb 1, 2018 · 9 comments · Fixed by #6448
Closed

feature request: read/write from/to samba (smb) drives #2042

dtenenba opened this issue Feb 1, 2018 · 9 comments · Fixed by #6448

Comments

@dtenenba
Copy link

dtenenba commented Feb 1, 2018

If rclone could support reading and writing from/to samba (smb) shares that would simplify my workflow greatly.
In other words, if rclone could do what smbclient does.

@ncw ncw added the new backend label Feb 3, 2018
@ncw ncw added this to the Help Wanted milestone Feb 3, 2018
@ncw
Copy link
Member

ncw commented Feb 3, 2018

If someone writes a go library to interface with smb then we'll do it, but writing a smb library is too big a project for us I think!

@dtenenba
Copy link
Author

dtenenba commented Feb 3, 2018

Thanks. Looks like there are a couple of packages out there, not sure if they are good enough:
https://github.com/stacktitan/smb
https://golanglibs.com/top?q=smb

@mathiasgredal
Copy link

Stacktitans implementation is very minimal and only includes the authentication parts. A library like https://github.com/videolabs/libdsm should most likely be ported to go before a backend could be implemented.

@ncw
Copy link
Member

ncw commented Nov 11, 2020

This looks like it might be complete enough: https://github.com/hirochachacha/go-smb2

@ivandeex
Copy link
Member

ivandeex commented Oct 23, 2021

Marking as important based on 15 👍 and availability of Go implementations.

@tooptoop4
Copy link

any idea when this will be available?

@ncw
Copy link
Member

ncw commented Sep 12, 2022

@tooptoop4 we are looking for someone to work on this. Do you want to have a go?

@tooptoop4
Copy link

there is a PR!!!

@ncw ncw closed this as completed in #6448 Sep 30, 2022
ncw pushed a commit that referenced this issue Sep 30, 2022
@ncw
Copy link
Member

ncw commented Oct 3, 2022

This is now merged to master.

If anyone would like to try it please use the latest beta and make new issues with any problems.

If all goes well it will be released in 1.60

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants