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

Deprecated notice using ContentRangeUploadHandler.php #156

Open
nico57r opened this issue Jun 22, 2023 · 2 comments
Open

Deprecated notice using ContentRangeUploadHandler.php #156

nico57r opened this issue Jun 22, 2023 · 2 comments

Comments

@nico57r
Copy link

nico57r commented Jun 22, 2023

What steps will reproduce the problem?

Enable deprecated notice log on php

What is the expected result?

No deprecated

What do you get instead?

preg_match(): Passing null to parameter #2 ($subject) of type string is deprecated in /var/www/instream-project-test.impactify.dev/build-9d9fc6b6/vendor/pion/laravel-chunk-upload/src/Handler/ContentRangeUploadHandler.php on line 113

Additional info

To resolv issue please change in

/vendor/pion/laravel-chunk-upload/src/Handler/ContentRangeUploadHandler.php

Line 70 :
$contentRange = $this->request->header(self::CONTENT_RANGE_INDEX);
To : $contentRange = $this->request->header(self::CONTENT_RANGE_INDEX, '');

Q A
This Package Version 1.5.2
Laravel Framework Version 10.9.0
PHP version 8.1.5
Operating system Ubuntu / Debian
@nico57r
Copy link
Author

nico57r commented Dec 22, 2023

no news ?

@pionl
Copy link
Owner

pionl commented Mar 25, 2024

Thanks to @RTippin is is completed. Thank you.

https://github.com/pionl/laravel-chunk-upload/releases/tag/v1.5.4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants