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] - Make SAMBA share optional #196

Open
ogarcia opened this issue Jun 21, 2023 · 4 comments
Open

[Feature Request] - Make SAMBA share optional #196

ogarcia opened this issue Jun 21, 2023 · 4 comments
Assignees
Labels
backlog Implementation/fix not planned yet new feature New feature or request

Comments

@ogarcia
Copy link

ogarcia commented Jun 21, 2023

Description

Right now you need to enter the SAMBA share to be able to connect to the remote, it would be interesting to make this optional and to list the shares in case you are not provided with any.

Changes

Make the share field of the form optional and in that case list the remote shares.

@ogarcia ogarcia added the new feature New feature or request label Jun 21, 2023
@0323pin
Copy link
Contributor

0323pin commented Jun 30, 2023

Actually, I would prefer if the whole smb module could be feature gated, veeso/pavao#5 (comment)

@veeso veeso added the backlog Implementation/fix not planned yet label Jul 6, 2023
@veeso
Copy link
Owner

veeso commented Jul 6, 2023

It doesn't look like libsmbclient has a function to list shares, or I cannot find it 🤷🏻

@veeso veeso added wontfix This will not be worked on and removed backlog Implementation/fix not planned yet labels Jul 6, 2023
@ogarcia
Copy link
Author

ogarcia commented Jul 6, 2023

It doesn't look like libsmbclient has a function to list shares, or I cannot find it 🤷🏻

I don't understand much about C, but this is the function that lists the shares in smbclient: https://github.com/samba-team/samba/blob/master/source4/client/client.c#L2684

Maybe with that information something can be done.

@veeso
Copy link
Owner

veeso commented Jul 6, 2023

Oook, it looks like it's only in SMB4. It could be a future development, but I'd need to think about how to display on the UI, it may be not that easy to handle tbh

@veeso veeso added backlog Implementation/fix not planned yet and removed wontfix This will not be worked on labels Jul 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backlog Implementation/fix not planned yet new feature New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants