Skip to content

Update TwoFactorAuth.php#116

Closed
motsmanish wants to merge 1 commit intoRobThree:masterfrom
motsmanish:patch-1
Closed

Update TwoFactorAuth.php#116
motsmanish wants to merge 1 commit intoRobThree:masterfrom
motsmanish:patch-1

Conversation

@motsmanish
Copy link

Deprecated function: rawurlencode(): Passing null to parameter #1

Deprecated function: rawurlencode(): Passing null to parameter RobThree#1
@willpower232
Copy link
Collaborator

Thanks for your attention but aren't those two variables type hinted by the function declaration so can only be strings?

@NicolasCARPi
Copy link
Collaborator

NicolasCARPi commented Nov 17, 2023

I agree with Will, this change makes no sense. Recommend closing this.

@RobThree
Copy link
Owner

Agree with all above.

@RobThree RobThree closed this Nov 17, 2023
@motsmanish
Copy link
Author

In one of our projects, we use usernames instead of emails. This led to an issue where the library received a null value for the label, causing the error: 'Deprecated function: rawurlencode(): Passing null to parameter'. But no worries, I’ll ensure to include more specific details next time I report an issue.

@willpower232
Copy link
Collaborator

@motsmanish is it possible you had an older version of the library which didn't have type hinting (see #97)?

@motsmanish
Copy link
Author

You are right, as per composer "robthree/twofactorauth": "^1.8.1".
Thank you for your time.

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

Successfully merging this pull request may close these issues.

4 participants