Information Required ? Pending

User tests: Successful: Unsuccessful:

avatar Razzo1987
Razzo1987
31 Aug 2020

Pull Request for Issue #30494 .

Summary of Changes

Change
<?php echo base64_decode($value->hash_id); ?> with <?php echo \base64_decode($value->hash_id); ?> to circumvent the recognition of "echo base64_decode" as a virus in cPanel (false positive).

Testing Instructions

  1. Try to upload in cPanel file manager the compiled zip file (NO PATCH TESTER) from https://ci.joomla.org/artifacts/joomla/joomla-cms/4.0-dev/30529/downloads/35140
  2. I don't know how to test default_updated_files.php functionality

Actual result BEFORE applying this Pull Request

  1. Uploading alert a Virus in the ZIP file:
    immagine
  2. Test of default_updated_files.php

Expected result AFTER applying this Pull Request

  1. Uploading success:
    immagine
  2. Test of default_updated_files.php

Documentation Changes Required

cPanel version affected

86.0.22 ?
88.0 ?

avatar Razzo1987 Razzo1987 - open - 31 Aug 2020
avatar Razzo1987 Razzo1987 - change - 31 Aug 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 31 Aug 2020
Category Administration com_templates
avatar Razzo1987 Razzo1987 - change - 31 Aug 2020
The description was changed
avatar Razzo1987 Razzo1987 - edited - 31 Aug 2020
avatar conconnl
conconnl - comment - 31 Aug 2020

I uploaded the current Nightly Build and the one created by your PR.
Both upload successful on the Joomla Sandbox server.
I can't seem to re-produce the mentioned issue.

image

avatar Razzo1987 Razzo1987 - change - 31 Aug 2020
The description was changed
avatar Razzo1987 Razzo1987 - edited - 31 Aug 2020
avatar Razzo1987
Razzo1987 - comment - 31 Aug 2020

@conconnl wich version of cPanel do you have? I'm using 86.0.22

immagine

avatar conconnl
conconnl - comment - 31 Aug 2020

image

avatar brianteeman
brianteeman - comment - 31 Aug 2020

if that bad check can be tricked by inserting a slash then its not just a bad check its a terrible check

avatar Razzo1987
Razzo1987 - comment - 31 Aug 2020

@brianteeman you're absolutely right!
What worries me is the amount of users who may be unable to install Joomla due to this problem ?

avatar Razzo1987 Razzo1987 - change - 31 Aug 2020
The description was changed
avatar Razzo1987 Razzo1987 - edited - 31 Aug 2020
avatar brianteeman
brianteeman - comment - 31 Aug 2020

To the best of my knowledge this is NOT a cpanel issue but related to your hosts implementation of http://www.rfxn.com/projects/linux-malware-detect/

avatar Quy
Quy - comment - 10 Sep 2020

Closing as this is not the correct fix. Please post in https://forum.joomla.org/ to see if others are experiencing the same issue. Thanks.

avatar Quy Quy - close - 10 Sep 2020
avatar Quy Quy - change - 10 Sep 2020
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2020-09-10 23:17:31
Closed_By Quy
Labels Added: Information Required ?

Add a Comment

Login with GitHub to post a comment