$this->app -> app() fixes ErrorException in Laravel 8

[fgd007]

When using this package in an upgraded from Laravel 7 to Laravel 8 project I bump into ErrorExceptions in TwoFactorAuthManager.php

When changing $this->app to app() it seems to work fine again.

Great package btw!

[michaeldzjap]

Hmmm... It seems they renamed $this->app to $this->container in Laravel 8 and introduced a dedicated $this->config member.

Your solution works, but I don't like the fact that the "magic" app() method now has to be called in each instance creation method, especially since the config information we need is already present as a (new) member.

So thanks for your PR, but I will probably go with another solution for this, which unfortunately would introduce a breaking change in this package and so would require a major version update. I will do this soon.

[michaeldzjap]

Proper support for Laravel 8 has been added in version 3. Should work for you now.

[fgd007]

Great that you found a better solution. I was looking in the Upgrade Guide for these kind of changes, but couldn't find any quickly. Your solution is the right one, thanks!