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

files_external:notify selfTest breaks first event #14441

Closed
ariselseng opened this issue Feb 28, 2019 · 0 comments
Closed

files_external:notify selfTest breaks first event #14441

ariselseng opened this issue Feb 28, 2019 · 0 comments
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap bug
Milestone

Comments

@ariselseng
Copy link
Member

Steps to reproduce

  1. Use official nextcloud image with smbclient installed
  2. Mount smb storage
  3. Track changes with occ files_external:notify ID -v
  4. run on a machine with the smb share mounted directly (without nextcloud):
$  touch SHARE/test.txt
$  touch SHARE/test2.txt

Expected behaviour

Both test.txt and test2.txt should be listed in the output of occ files_external:notify

Actual behaviour

Only test2.txt gets outputted (and registered in the database)

When I comment out this line it goes away and expected behaviour starts: https://github.com/nextcloud/server/blob/master/apps/files_external/lib/Command/Notify.php#L140

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap bug
Projects
None yet
Development

No branches or pull requests

2 participants