Closed (fixed)
Project:
Authenticator Login
Version:
2.0.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
23 Jan 2024 at 08:25 UTC
Updated:
10 Dec 2024 at 17:54 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
santhosh-kumar-rengasamy commentedComment #3
santhosh-kumar-rengasamy commentedPlease try this patch for QA generation.
Comment #4
santhosh-kumar-rengasamy commentedThe below patch has been created for the version (2.0.x).
Please verify.
Comment #5
ahmed.raza commentedI've tested this on fresh Drupal 10 setup and couldn't reproduce. It works as designed.
Need more details on your module and drupal version and steps to reproduce.
Tested this on module version 2.0.8
Drupal version 10.2.8
PHP version 8.1
Comment #6
ahmed.raza commentedComment #7
awolfey commentedI have the same problem. Without the image tag the raw png data is printed.
data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAA.....The patch fixes this for me.
Comment #8
awolfey commentedModule version: 2.0.8
Drupal: 10.1.4
PHP: 8.1
Comment #9
ahmed.raza commentedAgain, the issue is not reproducable to me, Checked on fresh drupal installation. The code
`$authenticator->getQr()` returns svg structure, your patch is wrong svg structure cannot be put into < img src="[svg_code_here]" >.
The issue is not reproducable in the first place. Both of you try reinstalling the module.
Comment #10
ahmed.raza commentedComment #11
awolfey commented@ahmed, it looks like you're expecting an svg, but I'm getting a png. What kind of image toolkit and settings are you using? PNG does work here < img src="[encoded_png_code_here]" >, which is why this patch works for me. Thanks.
I've updated the module to the latest version, and reinstalled it. Core is now 10.2.4.
Comment #12
blade_ukraine commentedHere is one extra solution that will work with both types of generated qr-code encoding png to base64 and svg.
Comment #13
blade_ukraine commentedComment #14
blade_ukraine commentedFixed 2 type qr codes display patch
Comment #15
blade_ukraine commented@awolfey In my case, the situation is worse. Locally, I have SVG, on the server, I have PNG. Based on that, I've created patch #14
Comment #16
blade_ukraine commentedComment #17
rajdip_755Comment #19
rajdip_755Hi all,
I have successfully replicated the issue and for solving it, applied the patch attached in #14 .
This patch successfully fixed the QR code generation issue. I'm attaching here the screenshots of the config form before and after applying the patch respectively.


Along with that, I've converted the patch as MR also.
Thanks !
Comment #21
rajdip_755Comment #23
ahmed.raza commentedComment #24
ahmed.raza commentedThose facing this issue, update the module to latest version.
@blade_ukraine, thanks for the patch its merged.