1.0.1

[rtuszik]

No description provided.

[sourcery-ai]

Hey there - I've reviewed your changes and they look great!

Prompt for AI Agents

Please address the comments from this code review:

## Individual Comments

### Comment 1
<location> `entrypoint.sh:13` </location>
<code_context>
+    echo "Updating photon group GID from $CURRENT_GID to $PGID"
+    groupmod -o -g "$PGID" photon
+    echo "Updating ownership of files from GID $CURRENT_GID to $PGID"
+    find / -group "$CURRENT_GID" -exec chgrp -h "$PGID" {} \; 2>/dev/null
+fi
+
</code_context>

<issue_to_address>
**suggestion (performance):** Running 'find /' may be slow and affect system files.

Restrict the 'find' command to specific directories to avoid unintended changes and improve execution speed.

```suggestion
    # Restrict find to specific directories to avoid system files and improve speed
    for dir in /home /opt /var; do
        if [ -d "$dir" ]; then
            find "$dir" -group "$CURRENT_GID" -exec chgrp -h "$PGID" {} \; 2>/dev/null
        fi
    done
```
</issue_to_address>

### Comment 2
<location> `entrypoint.sh:20` </location>
<code_context>
+    echo "Updating photon user UID from $CURRENT_UID to $PUID"
+    usermod -o -u "$PUID" photon
+    echo "Updating ownership of files from UID $CURRENT_UID to $PUID"
+    find / -user "$CURRENT_UID" -exec chown -h "$PUID" {} \; 2>/dev/null
+fi
+
</code_context>

<issue_to_address>
**🚨 issue (security):** Changing ownership recursively from root may have unintended consequences.

Limiting the ownership change to application-specific directories will help prevent system instability and security risks.
</issue_to_address>

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.}

[sourcery-ai]

suggestion (performance): Running 'find /' may be slow and affect system files.

Restrict the 'find' command to specific directories to avoid unintended changes and improve execution speed.

Suggested change

	find / -group "$CURRENT_GID" -exec chgrp -h "$PGID" {} \; 2>/dev/null
	# Restrict find to specific directories to avoid system files and improve speed
	for dir in /home /opt /var; do
	if [ -d "$dir" ]; then
	find "$dir" -group "$CURRENT_GID" -exec chgrp -h "$PGID" {} \; 2>/dev/null
	fi
	done

[sourcery-ai]

🚨 issue (security): Changing ownership recursively from root may have unintended consequences.

Limiting the ownership change to application-specific directories will help prevent system instability and security risks.

[socket-security]

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff	Package	Supply Chain Security	Vulnerability	Quality	Maintenance	License
	apprise@​1.9.3 ⏵ 1.9.4	+1
	requests@​2.32.4 ⏵ 2.32.5
	ty@​0.0.1a16 ⏵ 0.0.1a20
	ruff@​0.12.7 ⏵ 0.12.12					+31

View full report