upgraded google2fa (0.7->5.0)

[mx2s]

#3378

[welcome]

Congratulations on opening your first Pull Request, this is a momentous day for you and us! ✨
To help us out, please make sure that you've followed the below:

• Non-English translations should be submitted to https://translate.cachethq.io - Documentation should be submitted to https://docs.cachethq.io - Please make sure that all tests pass! - Don't worry about StyleCI, it'll fix itself on merge.

[accesslint]

There are accessibility issues in these changes.

[accesslint]

👏 You fixed the issue(s)! Great work.

[jbrooksuk]

Hey, thanks for opening this @MXSS. I'm unsure how I feel about requiring Imagemagick to be installed as well.

Is there an alternative?

[mx2s]

@jbrooksuk in recent versions of used package (pragmarx/google2fa) image generation is disabled that why I had to install additional package Bacon/QRCode which requires imagick

You can check package readme for more details:
https://github.com/antonioribeiro/google2fa

there's a few alternatives to that package but I'm not sure yet if we can get rid of imagick, I'll take a look

[jbrooksuk]

Thanks @MXSS!

GD is much easier to install and is typical of most installations that I've come across, whereas I find Imagemagick less common.

[jbrooksuk]

It looks like https://github.com/chillerlan/php-qrcode has SVG support and doesn't list GD or Imagemagick as a requirement in composer.json.

[mx2s]

@jbrooksuk done

[jbrooksuk]

Nice! Could you rebase this please and we'll merge it in?

[welcome]

Hooray! Your first Pull Request was merged, here's to many more 🚀