[S3] fclose(): supplied resource is not a valid stream resource #28496

pierreozoux · 2021-08-18T13:47:58Z

How to use GitHub

Please use the 👍 reaction to show that you are affected by the same issue.
Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
Subscribe to receive notifications on status change and new comments.

Looks similar to #23373

Steps to reproduce

Go to web UI
upload a somehow big file, bigger than the chunk size that is by default 10MB, so for instance 100MB

Expected behaviour

There shouldn't be errors in the nextcloud.log

Actual behaviour

There are errors.

Server configuration

Nextcloud version: 20.0.11

Where did you install Nextcloud from: docker

Are you using external storage, if yes which one: ceph rados gateway / s3 compatible

Are you using encryption: no

Logs

Nextcloud log (data/nextcloud.log)

Nextcloud log

{
   "reqId":"RgbGEzK8IUMkYzSQjFY3",
   "level":3,
   "time":"2021-08-18T07:42:32+00:00",
   "remoteAddr":"49.12.19.171",
   "user":"admin",
   "app":"PHP",
   "method":"PUT",
   "url":"/remote.php/dav/uploads/admin/web-file-upload-761e5de13b91cf0c4ab6f34b6164e593-1629271135519/754974720",
   "message":{
      "Exception":"Error",
      "Message":"fclose(): supplied resource is not a valid stream resource at /usr/src/nextcloud/3rdparty/icewind/streams/src/Wrapper.php#132",
      "Code":0,
      "Trace":[
         {
            "function":"onError",
            "class":"OC\\Log\\ErrorHandler",
            "type":"::",
            "args":[
               2,
               "fclose(): supplied resource is not a valid stream resource",
               "/usr/src/nextcloud/3rdparty/icewind/streams/src/Wrapper.php",
               132,
               [
                  
               ]
            ]
         },
         {
            "file":"/usr/src/nextcloud/3rdparty/icewind/streams/src/Wrapper.php",
            "line":132,
            "function":"fclose",
            "args":[
               null
            ]
         },
         {
            "file":"/usr/src/nextcloud/3rdparty/icewind/streams/src/CallbackWrapper.php",
            "line":119,
            "function":"stream_close",
            "class":"Icewind\\Streams\\Wrapper",
            "type":"->",
            "args":[
               
            ]
         },
         {
            "function":"stream_close",
            "class":"Icewind\\Streams\\CallbackWrapper",
            "type":"->",
            "args":[
               
            ]
         },
         {
            "file":"/usr/src/nextcloud/3rdparty/guzzlehttp/psr7/src/Stream.php",
            "line":105,
            "function":"fclose",
            "args":[
               null
            ]
         },
         {
            "file":"/usr/src/nextcloud/3rdparty/guzzlehttp/psr7/src/Stream.php",
            "line":73,
            "function":"close",
            "class":"GuzzleHttp\\Psr7\\Stream",
            "type":"->",
            "args":[
               
            ]
         },
         {
            "function":"__destruct",
            "class":"GuzzleHttp\\Psr7\\Stream",
            "type":"->",
            "args":[
               
            ]
         }
      ],
      "File":"/usr/src/nextcloud/lib/private/Log/ErrorHandler.php",
      "Line":91,
      "CustomMessage":"--"
   },
   "userAgent":"Mozilla/5.0 (Macintosh; Intel Mac OS X 10.14; rv:92.0) Gecko/20100101 Firefox/92.0",
   "version":"20.0.11.1"
}

The text was updated successfully, but these errors were encountered:

juliusknorr · 2021-08-18T16:40:34Z

Should have been fixed by icewind1991/Streams@93bce47 but seems the package didn't get an update on the stable branches in the 3rdparty repo.

juliusknorr · 2021-08-18T16:41:44Z

@skjnldsv @ChristophWurst Should we maybe also setup dependabot for minor updates in the stable branches of https://github.com/nextcloud/3rdparty ?

ChristophWurst · 2021-08-18T17:27:38Z

Sure :)

PVince81 · 2021-10-08T07:32:03Z

also just reproduced on 21.0.5 with minio object store docker:

run docker run -p 9000:9000 minio/minio server /data
setup NC
edit config.php and set the settings like below

'objectstore' =>
	array (
		'class' => 'OC\\Files\\ObjectStore\\S3',
		'arguments' =>
		array (
			'bucket' => 'nextcloud-dev',
			'key' => 'minioadmin',
			'secret' => 'minioadmin',
			'hostname' => 'localhost',
			'port' => '9000',
			'use_ssl' => false,
			'use_path_style' => true,
		),
	),

login as admin/admin

I see this in the log:

{"reqId":"my0o14wBcN7ZTAbx1Obt","level":3,"time":"2021-10-08T07:27:49+00:00","remoteAddr":"127.0.0.1","user":"admin","app":"PHP","method":"GET","url":"/index.php/apps/files/","message":{"Exception":"Error","Message":"fclose(): supplied resource is not a valid stream resource at /srv/www/htdocs/server/3rdparty/icewind/streams/src/Wrapper.php#96","Code":0,"Trace":[{"function":"onError","class":"OC\\Log\\ErrorHandler","type":"::","args":[2,"fclose(): supplied resource is not a valid stream resource","/srv/www/htdocs/server/3rdparty/icewind/streams/src/Wrapper.php",96,[]]},{"file":"/srv/www/htdocs/server/3rdparty/icewind/streams/src/Wrapper.php","line":96,"function":"fclose","args":[null]},{"file":"/srv/www/htdocs/server/3rdparty/icewind/streams/src/CallbackWrapper.php","line":117,"function":"stream_close","class":"Icewind\\Streams\\Wrapper","type":"->","args":[]},{"function":"stream_close","class":"Icewind\\Streams\\CallbackWrapper","type":"->","args":[]},{"file":"/srv/www/htdocs/server/3rdparty/guzzlehttp/psr7/src/Stream.php","line":108,"function":"fclose","args":[null]},{"file":"/srv/www/htdocs/server/3rdparty/guzzlehttp/psr7/src/Stream.php","line":74,"function":"close","class":"GuzzleHttp\\Psr7\\Stream","type":"->","args":[]},{"function":"__destruct","class":"GuzzleHttp\\Psr7\\Stream","type":"->","args":[]}],"File":"/srv/www/htdocs/server/lib/private/Log/ErrorHandler.php","Line":92,"CustomMessage":"--"},"userAgent":"Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/93.0.4577.82 Safari/537.36","version":"21.0.5.1"}

PVince81 · 2021-10-08T09:35:48Z

update 3rdparty lib: Bump icewind/streams from 0.7.2 to 0.7.5 3rdparty#730
update 3rdparty pointer: [stable21] 3rdparty: Bump icewind/streams from 0.7.2 to 0.7.5 #28505
update icewind/streams in files_external: [stable21] Update icewind/streams to 0.7.5 in files_external #29137

PVince81 · 2021-10-18T12:57:46Z

Reponing to track backport to stable20 because I need somewhere to put the long checklist:

stable20: update 3rdparty lib Bump icewind/streams from 0.7.1 to 0.7.5 3rdparty#855
stable20: update 3rdparty pointer Update 3rdparty ref #29297
stable20: update icewind/streams in files_external Bump icewind/streams from 0.7.3 to 0.7.5 in files_external #29298

kesselb · 2021-10-22T20:30:07Z

PHP Warning: Declaration of OC\Files\Stream\Encryption::loadContext($name) should be compatible with Icewind\Streams\Wrapper::loadContext($name = NULL) in /drone/src/lib/private/Files/Stream/Encryption.php on line 0

PHP Warning: Declaration of OC\Files\Stream\Encryption::wrapSource($source, $context, $protocol, $class, $mode = 'r+') should be compatible with Icewind\Streams\WrapperHandler::wrapSource($source, $context = Array, $protocol = NULL, $class = NULL, $mode = 'r+') in /drone/src/lib/private/Files/Stream/Encryption.php on line 0

Found the above error messages in a backport for stable20: https://drone.nextcloud.com/nextcloud/server/11030/14/5

The update from 0.7.1 to 0.7.2 for stable21 required some code changes: #24533

Unfortunately the 0.7.2 update and the code changes were not backported to stable20.

PVince81 · 2021-10-25T07:12:05Z

backports are merged, closing

PVince81 · 2021-10-25T07:20:20Z

PHP Warning: Declaration of OC\Files\Stream\Encryption::loadContext($name) should be compatible with Icewind\Streams\Wrapper::loadContext($name = NULL) in /drone/src/lib/private/Files/Stream/Encryption.php on line 0

PHP Warning: Declaration of OC\Files\Stream\Encryption::wrapSource($source, $context, $protocol, $class, $mode = 'r+') should be compatible with Icewind\Streams\WrapperHandler::wrapSource($source, $context = Array, $protocol = NULL, $class = NULL, $mode = 'r+') in /drone/src/lib/private/Files/Stream/Encryption.php on line 0

Found the above error messages in a backport for stable20: https://drone.nextcloud.com/nextcloud/server/11030/14/5

The update from 0.7.1 to 0.7.2 for stable21 required some code changes: #24533

Unfortunately the 0.7.2 update and the code changes were not backported to stable20.

oops, I'll recheck

PVince81 · 2021-10-25T07:38:34Z

PR here for the missing code changes for stable20: #29426

skjnldsv · 2021-10-25T12:25:54Z

PR here for the missing code changes for stable20: #29426

Merged

pierreozoux added bug 0. Needs triage Pending check for reproducibility or if it fits our roadmap labels Aug 18, 2021

juliusknorr added 1. to develop Accepted and waiting to be taken care of feature: object storage and removed 0. Needs triage Pending check for reproducibility or if it fits our roadmap labels Aug 18, 2021

solracsf linked a pull request Aug 20, 2021 that will close this issue

[stable21] 3rdparty: Bump icewind/streams from 0.7.2 to 0.7.5 #28505

Merged

2 tasks

solracsf mentioned this issue Aug 20, 2021

[stable21] 3rdparty: Bump icewind/streams from 0.7.2 to 0.7.5 #28505

Merged

2 tasks

PVince81 self-assigned this Oct 8, 2021

PVince81 mentioned this issue Oct 8, 2021

[stable21] Update icewind/streams to 0.7.5 in files_external #29137

Merged

PVince81 closed this as completed Oct 8, 2021

PVince81 reopened this Oct 18, 2021

This was referenced Oct 18, 2021

Bump icewind/streams from 0.7.1 to 0.7.5 nextcloud/3rdparty#854

Closed

Bump icewind/streams from 0.7.1 to 0.7.5 nextcloud/3rdparty#855

Merged

Update 3rdparty ref #29297

Merged

Bump icewind/streams from 0.7.3 to 0.7.5 in files_external #29298

Merged

PVince81 closed this as completed Oct 19, 2021

kesselb reopened this Oct 22, 2021

PVince81 closed this as completed Oct 25, 2021

PVince81 reopened this Oct 25, 2021

PVince81 mentioned this issue Oct 25, 2021

Extra fixes needed for icewind/streams update to 0.7.2 #29426

Merged

1 task

skjnldsv closed this as completed Oct 25, 2021

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[S3] fclose(): supplied resource is not a valid stream resource #28496

[S3] fclose(): supplied resource is not a valid stream resource #28496

pierreozoux commented Aug 18, 2021

juliusknorr commented Aug 18, 2021

juliusknorr commented Aug 18, 2021

ChristophWurst commented Aug 18, 2021

PVince81 commented Oct 8, 2021

PVince81 commented Oct 8, 2021 •

edited

Loading

PVince81 commented Oct 18, 2021 •

edited

Loading

kesselb commented Oct 22, 2021

PVince81 commented Oct 25, 2021

PVince81 commented Oct 25, 2021

PVince81 commented Oct 25, 2021

skjnldsv commented Oct 25, 2021

[S3] fclose(): supplied resource is not a valid stream resource #28496

[S3] fclose(): supplied resource is not a valid stream resource #28496

Comments

pierreozoux commented Aug 18, 2021

How to use GitHub

Steps to reproduce

Expected behaviour

Actual behaviour

Server configuration

Logs

Nextcloud log (data/nextcloud.log)

juliusknorr commented Aug 18, 2021

juliusknorr commented Aug 18, 2021

ChristophWurst commented Aug 18, 2021

PVince81 commented Oct 8, 2021

PVince81 commented Oct 8, 2021 • edited Loading

PVince81 commented Oct 18, 2021 • edited Loading

kesselb commented Oct 22, 2021

PVince81 commented Oct 25, 2021

PVince81 commented Oct 25, 2021

PVince81 commented Oct 25, 2021

skjnldsv commented Oct 25, 2021

PVince81 commented Oct 8, 2021 •

edited

Loading

PVince81 commented Oct 18, 2021 •

edited

Loading