rdt-client icon indicating copy to clipboard operation
rdt-client copied to clipboard

Fix permissions sweep

Open kode54 opened this issue 8 months ago • 10 comments

User description

It should not be setting permissions on data that lives in the image.

Fixes #716


PR Type

Bug fix


Description

  • Restricts permissions setting to /data directory only

  • Prevents unnecessary chown on /app during container startup

  • Addresses Docker container startup hang issue


Changes walkthrough 📝

Relevant files
Bug fix
run
Restrict chown operation to /data directory only                 

root/etc/s6-overlay/s6-rc.d/init-rdt-client/run

  • Removes /app from the recursive chown command
  • Now only /data has its ownership set to abc:abc
  • Prevents permission changes on image-internal files
  • +1/-1     

    Need help?
  • Type /help how to ... in the comments thread for any questions about Qodo Merge usage.
  • Check out the documentation for more information.
  • kode54 avatar Feb 22 '25 09:02 kode54