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

RawConnection.php: add workgroup to auth file #34

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

aklomp
Copy link

@aklomp aklomp commented Aug 23, 2015

I read somewhere in a discussion about libsmbclient-php that people were having trouble with SMB and workgroups when using the smbclient binary. Maybe it's because the workgroup parameter is not being passed to smbclient in the auth file? This pull request, which is based on similar code in SambaDAV, adds support for that parameter. Beware that I haven't actually tested or ran this code.

@icewind1991
Copy link
Owner

From what I understood after some googling (haven't been able to reproduce the original problem to test) is that the domain option in the auth file is pretty much ignored and that you should set the -W cli param instead (should be in the next 8.1 bugfix release).

@icewind1991 icewind1991 force-pushed the master branch 2 times, most recently from 6889d88 to 82f9618 Compare August 26, 2016 22:44
@icewind1991 icewind1991 force-pushed the master branch 2 times, most recently from e1438ac to 6691355 Compare August 16, 2017 16:09
@icewind1991 icewind1991 force-pushed the master branch 4 times, most recently from eda3870 to 29bdeba Compare March 13, 2018 18:06
@icewind1991 icewind1991 force-pushed the master branch 3 times, most recently from c1a84fc to 0fc2a08 Compare August 28, 2018 14:35
@icewind1991 icewind1991 force-pushed the master branch 7 times, most recently from e4b0bcd to 80a4edf Compare March 2, 2021 17:56
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 this pull request may close these issues.

2 participants