Skip to content

additional debug logging on smb copy/rename failures - #7555

Merged
icewind1991 merged 1 commit into
masterfrom
smb-copy-rename-log
Dec 18, 2017
Merged

additional debug logging on smb copy/rename failures#7555
icewind1991 merged 1 commit into
masterfrom
smb-copy-rename-log

Conversation

@icewind1991

Copy link
Copy Markdown
Member

Signed-off-by: Robin Appelman robin@icewind.nl

Signed-off-by: Robin Appelman <robin@icewind.nl>
@icewind1991icewind1991 added the 3. to review Waiting for reviews label Dec 18, 2017
@icewind1991icewind1991 added this to the Nextcloud 13 milestone Dec 18, 2017
@blizzzblizzz added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Dec 18, 2017
@codecov

codecovBot commented Dec 18, 2017

Copy link
Copy Markdown

Codecov Report

Merging #7555 into master will decrease coverage by 0.02%.
The diff coverage is 33.33%.

@@ Coverage Diff @@## master #7555 +/- ##
============================================
- Coverage 51.17% 51.14% -0.03% - Complexity 24869 24875 +6 
============================================
Files 1601 1601 Lines 94692 94736 +44 Branches 1368 1368 ============================================
- Hits 48454 48450 -4 - Misses 46238 46286 +48
Impacted FilesCoverage ΔComplexity Δ
apps/files_external/lib/Lib/Storage/SMB.php5.63% <0%> (-0.03%)116 <0> (ø)
lib/private/Files/Storage/Common.php69.2% <50%> (-0.11%)147 <0> (+1)
...private/Collaboration/Collaborators/MailPlugin.php38.84% <0%> (-19.91%)24% <0%> (+5%)
core/js/js.js62.98% <0%> (-0.57%)0% <0%> (ø)

@icewind1991
icewind1991 merged commit bdf4111 into masterDec 18, 2017
@icewind1991
icewind1991 deleted the smb-copy-rename-log branch December 18, 2017 14:02
@MorrisJobkeMorrisJobke mentioned this pull request Jan 2, 2018
30 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to releaseReady to be released and/or waiting for tests to finish

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@icewind1991@MorrisJobke@blizzz