-
-
Notifications
You must be signed in to change notification settings - Fork 600
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
Undefined method SplFileObject::fread() #256
Comments
I don't think that we should fix bugs for EOL'd PHP versions, so I'm closing as |
Well, you should at least update composer.json with minimum php version 5.5.11 for this version so you don't lie about which php version is needed. |
True that. @lcobucci what's your opinion on that? Reopen and fix? EOL our
version?
…On Thu, 2 Aug 2018, 23:05 Trond Aasan, ***@***.***> wrote:
Well, you should at least update composer.json with minimum php version
5.5.11 for this version so you don't lie about which php version is needed.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
<#256 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AAJakJ3TJteKaNi-HvQ-gsJ_4KmvrI09ks5uM2msgaJpZM4VsQSH>
.
|
@taasan I'm very sorry for that issue, it's been like ages since I use that version of PHP that I completely forgot about it. Making the change compatible with your PHP version is so simple that I rather modify the library. But please try to upgrade your stack as soon as possible, v3.3 will require PHP 5.6 and v4 PHP 7.2 (possibly 7.3) |
Thanks :-) I am unfortunately stuck with Ubuntu 14.04 for another year |
Commit 8e80bec introduced an error on PHP 5.5.9 (which I am stuck at)
PHP Fatal error: Call to undefined method SplFileObject::fread() in /var/www/vendor/lcobucci/jwt/src/Signer/Key.php on line 66
The text was updated successfully, but these errors were encountered: