The current 7.x-1.x-dev includes a number of files with wrong permissions as 755 rather than 644.

Thanks.

CommentFileSizeAuthor
#3 eu_cookie_compliance-wrong-file-permissions-3009862-3-7.x.patch1.17 KBsvenryen

Comments

izmeez created an issue. See original summary.

izmeez’s picture

izmeez commented
Title: Wrong file permissions in » Wrong file permissions in 7.x-1.x-dev
svenryen’s picture

svenryen commented
StatusFileSize
new eu_cookie_compliance-wrong-file-permissions-3009862-3-7.x.patch1.17 KB

Hi @izmeez, thanks for reporting that.

Can you help review the patch and see if it covers all files, or if I missed any?

izmeez’s picture

izmeez commented

Yes, the files in the patch match what I found on my testing, only eu_cookie_compliance.info was not in my list.

jacob.embree’s picture

jacob.embree commented
Status: Active » Reviewed & tested by the community

#3 covers all the files and is valid for both 7.x-1.x and 7.x-2.x.

  • svenryen committed 41beaf9 on 7.x-1.x 
    Issue #3009862 by svenryen, izmeez: Wrong file permissions in 7.x-1.x-...

  • svenryen committed 34ad05d on 7.x-2.x 
    Merge branch '7.x-1.x' into 7.x-2.x

* 7.x-1.x: (27 commits)
Issue #...
  • svenryen committed 41beaf9 on 7.x-2.x
    Issue #3009862 by svenryen, izmeez: Wrong file permissions in 7.x-1.x-...
svenryen’s picture

svenryen commented
Status: Reviewed & tested by the community » Fixed

Awesome, I just committed the patch to -dev.

  • svenryen committed 34ad05d on 7.x-1.x 
    Merge branch '7.x-1.x' into 7.x-2.x

* 7.x-1.x: (27 commits)
Issue #...

Status: Fixed » Closed (fixed)

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