clientside_validation_jquery/clientside_validation_jquery.libraries.yml file has "execute" permission and checkouts with permissions -rwxr-xr-x (755).
I believe it should not have "execute" permission and should checkout with permissions -rw-r--r-- (644).

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

asvira created an issue. See original summary.

asvira’s picture

  • nikunjkotecha committed 9188dcd on 8.x-2.x authored by asvira
    Issue #3073528 by asvira: clientside_validation_jquery.libraries.yml has...
nikunjkotecha’s picture

Status: Needs review » Fixed

Thanks @asvira for reporting and the patch.

Just curious, is there any issue with this permission? Any doc or reference you went through to find this as a vulnerability or issue?

asvira’s picture

@nikunjkotecha, I don't think there is any real issue, except that human commits the file as non-executable (just because it should be so) and then composer makes file executable (when installing/updating/patching) and it produces additional line about modified file in git status (which requires attention each time).

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.