diff options
| author | Nils Adermann <naderman@naderman.de> | 2014-10-27 19:04:01 -0700 |
|---|---|---|
| committer | Nils Adermann <naderman@naderman.de> | 2014-10-27 19:04:01 -0700 |
| commit | b9c08cddd47b39355458e828d847c34ab0e8c378 (patch) | |
| tree | 87460fd5cc43eb75ecce8c447a9eaa4e25a01cfc /phpBB/phpbb/avatar | |
| parent | 4c75f013de1040bf74886de1c1a6d4b83e2e8453 (diff) | |
| parent | efe3aaaf0b9b2f60da89df0e285dace91f24cea1 (diff) | |
| download | forums-b9c08cddd47b39355458e828d847c34ab0e8c378.tar forums-b9c08cddd47b39355458e828d847c34ab0e8c378.tar.gz forums-b9c08cddd47b39355458e828d847c34ab0e8c378.tar.bz2 forums-b9c08cddd47b39355458e828d847c34ab0e8c378.tar.xz forums-b9c08cddd47b39355458e828d847c34ab0e8c378.zip | |
Merge branch 'prep-release-3.1.0' into develop-ascraeus
* prep-release-3.1.0:
[ticket/13227] Properly handle curlwrapper headers in remote avatars
[ticket/13231] Update the dependencies of the migration contact_admin_form
Diffstat (limited to 'phpBB/phpbb/avatar')
| -rw-r--r-- | phpBB/phpbb/avatar/driver/remote.php | 18 |
1 files changed, 17 insertions, 1 deletions
diff --git a/phpBB/phpbb/avatar/driver/remote.php b/phpBB/phpbb/avatar/driver/remote.php index 1bd7f6c927..4b0ee3f06f 100644 --- a/phpBB/phpbb/avatar/driver/remote.php +++ b/phpBB/phpbb/avatar/driver/remote.php @@ -130,8 +130,24 @@ class remote extends \phpbb\avatar\driver\driver { // Timeout after 1 second stream_set_timeout($file_stream, 1); + // read some data to ensure headers are present + fread($file_stream, 1024); $meta = stream_get_meta_data($file_stream); - foreach ($meta['wrapper_data'] as $header) + + if (isset($meta['wrapper_data']['headers']) && is_array($meta['wrapper_data']['headers'])) + { + $headers = $meta['wrapper_data']['headers']; + } + else if (isset($meta['wrapper_data']) && is_array($meta['wrapper_data'])) + { + $headers = $meta['wrapper_data']; + } + else + { + $headers = array(); + } + + foreach ($headers as $header) { $header = preg_split('/ /', $header, 2); if (strtr(strtolower(trim($header[0], ':')), '_', '-') === 'content-type') |