laravel-otp

laravel-otp copied to clipboard

Hi there @ichtrojan I'd like to know if this package supports Laravel 9? I try to follow the installation step, but the package seems not discoverable. I find a number...

sluxzer

Error: Using $this when not in object context

1

Hello, When I'm trying to generate the OTP using this method `Ichtrojan\Otp\Otp::generate($email, 6, 15);` I'm getting this error Error: Using $this when not in object context in file /vendor/ichtrojan/laravel-otp/src/Otp.php on...

tomerof

okedialf

Also replaced `$this->generatePin` with `self::generatePin`

okedialf

Non-static method Ichtrojan\\Otp\\Otp::generate() cannot be called statically

1

This was the error I got after adding the package to my laravel, **Non-static method Ichtrojan\\Otp\\Otp::generate() cannot be called statically**

dtobi59

Hi. This PR - completely separates concerns (Otp Class, Facades) - Autoloads the Otp facade as well. - also fixing #11 - A Lil documentation, some .gitignore cleanups.

yondifon

Going through Otp.php in this line https://github.com/ichtrojan/laravel-otp/blob/6ba01e246cbc35322180f3e16d8d7a24ce378363/src/Otp.php#L106, I noticed you are using the mt_rand function. this is not a secure random generator function as seen from the docs here https://www.php.net/manual/en/function.mt-rand.php....

haibeey

Add static method new for creating instances in Otp class This allows for a more readable, such as `Otp::new()->generate('[email protected]', 'numeric', 6, 15); ` instead of `(new Otp)->generate('[email protected]', 'numeric', 6, 15);`

moecods

## Pull Request Description This pull request introduces the following changes: 1. Refactored the hard-coded configuration of validity duration and the length of the token. Now, these configurations are set...

shahsawoodshinwari

Is there any way to change the default table name from otps to tbl_otps

fanfigo