JWTRefreshTokenBundle Deprecation warning: Implicitly marking parameter $authException as nullable is deprecated, the explicit nullable type must be used instead

Deprecation warning: Implicitly marking parameter $authException as nullable is deprecated, the explicit nullable type must be used instead

Open BartekDe opened this issue 10 months ago • 18 comments

When using this package with PHP 8.4 a deprecation warning is emitted when the \Gesdinet\JWTRefreshTokenBundle\Security\Http\Authenticator\RefreshTokenAuthenticator::start function is called.

Gesdinet\JWTRefreshTokenBundle\Security\Http\Authenticator\RefreshTokenAuthenticator:: start(): Implicitly marking parameter $authException as nullable is deprecated, the explicit nullable type must be used instead in <b>.../vendor/gesdinet/jwt-refresh-token-bundle/Security/Http/Authenticator/RefreshTokenAuthenticator.php</b> on line <b>181</b><br />

Feb 01 '25 21:02 BartekDe

I see that there are 2 pull requests already opened that fix this issue.

@markitosgv do you plan to merge this soon? https://github.com/markitosgv/JWTRefreshTokenBundle/pull/389

Feb 01 '25 21:02 BartekDe

Feb 04 '25 12:02 ViPErCZ

It has now been merged :) Waiting for a future release now

Feb 11 '25 08:02 froozeify

Is it possible to have a tag with this fix?

Mar 11 '25 07:03 XanderEVG

Is it possible to have a tag with this fix?

Mar 28 '25 06:03 s3bul

Is it possible to have a tag with this fix?

Apr 01 '25 08:04 Eskaaa

Apr 04 '25 08:04 samius

Apr 04 '25 08:04 stehled

Apr 04 '25 08:04 mikusekdan

Apr 04 '25 08:04 DanHariton

Apr 04 '25 08:04 TomasCzech

Apr 04 '25 10:04 domPatera

If it might be useful, I've created a patch to be used in the meantime, until a tag is published or a new release is created: https://gist.github.com/LucianoVandi/5fc0b9e6bfb5155e69ddcd019cbd4a3e