- Issue created by @Santhosh-Kumar-Rengasamy
- 🇮🇳India Santhosh-Kumar-Rengasamy
The below patch has been created for the version (2.0.x).
Please verify. - Status changed to Postponed: needs info
9 months ago 7:07am 20 February 2024 - 🇵🇰Pakistan Ahmed.Raza
I'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.
- Status changed to RTBC
8 months ago 9:10pm 18 March 2024 - 🇺🇸United States awolfey
I 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. - 🇵🇰Pakistan Ahmed.Raza
Again, 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 .
The issue is not reproducable in the first place. Both of you try reinstalling the module.
- Status changed to Closed: works as designed
8 months ago 2:46pm 31 March 2024 - 🇺🇸United States awolfey
@ahmed, it looks like you're expecting an svg, but I'm getting a png. What kind of image toolkit are you using? PNG does work here < img src="[encoded_png_code_here]" >, which is why this patch works for me. Thanks.
- 🇺🇦Ukraine blade_ukraine
Here is one extra solution that will work with both types of generated qr-code encoding png to base64 and svg.
- 🇺🇦Ukraine blade_ukraine
@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
- 🇮🇳India rajdip_755 kolkata
rajdip_755 → changed the visibility of the branch 3416397-qr-code-is to hidden.
- 🇮🇳India rajdip_755 kolkata
Hi all,
I have successfully replicated the issue and for solving it, applied the patch attached in #14 .
This patch is 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 !
- @rajdip_755 opened merge request.