Skip to content

Make sure invalid images don't flood the log file#3439

Merged
LukasReschke merged 2 commits intomasterfrom
oc-26963
Feb 16, 2017
Merged

Make sure invalid images don't flood the log file#3439
LukasReschke merged 2 commits intomasterfrom
oc-26963

Conversation

@nickvergessen
Copy link
Member

Upstream 26963

Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen added the 3. to review Waiting for reviews label Feb 10, 2017
@nickvergessen nickvergessen added this to the Nextcloud 12.0 milestone Feb 10, 2017
@mention-bot
Copy link

@nickvergessen, thanks for your PR! By analyzing the history of the files in this pull request, we identified @tanghus, @Kondou-ger and @butonic to be potential reviewers.

Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
@blizzz
Copy link
Member

blizzz commented Feb 10, 2017

There was 1 failure:

  1. OCA\Files_External\Tests\Storage\SmbTest::testNotifyGetChanges
    Failed asserting that an array contains OC\Files\Notify\RenameChange Object &00000000413754ab000000005176f3dc (
    'targetPath' => 'renamed.txt'
    'type' => 4
    'path' => 'newfile.txt'
    ).
    /drone/src/github.com/nextcloud/server/apps/files_external/tests/Storage/SmbTest.php:114
    FAILURES!
    Tests: 84, Assertions: 292, Failures: 1.

Does not seem related … but was this failing (sporadically) earlier?

@LukasReschke
Copy link
Member

Does not seem related … but was this failing (sporadically) earlier?

@icewind1991 Any idea about this? :)

@icewind1991
Copy link
Member

Any idea about this? :)

Ignore it for now, will see if I can make that test more reliable later

@LukasReschke LukasReschke merged commit cf582a6 into master Feb 16, 2017
@LukasReschke LukasReschke deleted the oc-26963 branch February 16, 2017 15:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants