Open paulocoghi opened 3 years ago
I am at your total disposal to provide every additional information needed.
Root cause found When an editor inserts a link to a related article, he can, sometimes, wrongly choose the title instead of the URL. And this is the cause for the malformed URL and the error on the AMP version.
Suggestion: Since the lack of availability of the AMP version represents loss of visits (millions, in this customer case), I would suggest to provide an option to enable the removal of a malformed link and to compile the AMP without it, instead of not compiling it.
In this customer scenario, such feature would be positive, since its editors are mostly non-programmers and non-developers.
You can close this issue, if you want to. Thanks again!
Edit: New information on the end of this issue! We found the root cause!
Versions: Wordpress - 5.7.2 Better AMP - 1.12.0
Ubuntu - 20.04.2 Nginx - 1.19.9 PHP - 7.4 FPM MariaDB - 10.5 (but the error also occurred when we used MySQL 8 instead of MariaDB)
Theme: Publisher (by Better Studio) - 7.9.2
Issue: In the last few months, some AMP pages created by Better AMP present the error below:
Observations:
http://The Page Title
Example URL with error (but with debug disabled) https://www.dci.com.br/dci-mais/noticias/filme-do-sbt-hoje-11-6-a-princesa-xuxa-e-os-trapalhoes/143275/amp/
Example with working URL https://www.dci.com.br/investimentos/criptomoedas/twitter-pode-ser-primeira-grande-rede-social-a-integrar-o-bitcoin/143591/amp/
Origin: https://github.com/better-studio/better-amp/blob/fb7bdb0540095a4e8f390f0cebd73f5355fc0b37/includes/functions/utility-function.php#L196-L228
What extra information do you consider important that I mention, in addition to the information above?