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

sftp-server: Add support for BSD style advisory file locking operations #66

Closed
wants to merge 2 commits into from

Conversation

kalvdans
Copy link

Only non-blocking shared or exclusive locks are supported for now. Allocate a new extension string for the flock operations.

…OCK_NB to sftp-server. Allocate a new extension string for the flock operations.
@kalvdans
Copy link
Author

The corresponding changes in sshfs that takes advantage of this protocol extension can be viewed at: libfuse/sshfs#55

@estan
Copy link

estan commented Jan 29, 2018

We're waiting eagerly for this feature. It would be great if a maintainer could take a look at this PR.

@Jakuje
Copy link
Contributor

Jakuje commented Jan 31, 2018

If you are interested in these changes in OpenSSH, you should fill the bug report on http://bugzilla.mindrot.org/ -- the github is used only as a mirror, as already said many times.

@estan
Copy link

estan commented Jan 31, 2018

@Jakuje Aha, did not know that, sorry for the noise. @kalvdans would you mind filing a bug with the patch at Bugzilla?

@kalvdans
Copy link
Author

Thanks @Jakuje, a bug has been posted to https://bugzilla.mindrot.org/show_bug.cgi?id=2828

@djmdjm
Copy link
Contributor

djmdjm commented Feb 10, 2018

Sorry, but we have no desire to add file locking to sftp/sftp-server. In fact, addition of file locking feature to the sftp protocol was why we never implemented versions of the protocol after 3.

@djmdjm
Copy link
Contributor

djmdjm commented Feb 10, 2018

Also, please use https://bugzilla.mindrot.org/ instead of github PRs in future. We only use this site as a mirror and seldom check the PR queue.

@djmdjm djmdjm closed this Feb 10, 2018
@kalvdans
Copy link
Author

Thanks @djmdjm for giving a definitive answer, I've closed the bug reports referenced by this PR as well now.

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