[PR PATCH] Update run permission diring db creation

[PR PATCH] Update run permission diring db creation

[Undersk0re]

[-- Attachment #1: Type: text/plain, Size: 846 bytes --]

There is a new pull request by Undersk0re against master on the void-packages repository

https://github.com/Undersk0re/void-packages patch-1
https://github.com/void-linux/void-packages/pull/53809

Update run permission diring db creation
Fix permissions in `/tmp` folder to prevent initialization timeout for PostgreSQL

This change resolves an issue where PostgreSQL initialization fails with the following error:
"down: postgresql16: 0s, normally up, want up; run: log: (pid 1234) 23s"

# Testing the changes
####    Was this tested? YES
-  This is a minor fix, and it has been verified to address the issue.
-  Security note: Permissions in `/tmp` can be sensitive; please review this commit to ensure it complies with your security policies.

A patch file from https://github.com/void-linux/void-packages/pull/53809.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-patch-1-53809.patch --]
[-- Type: text/x-diff, Size: 969 bytes --]

From fb3922bdbbacf3aaac285e3e43d1a694d80d94a1 Mon Sep 17 00:00:00 2001
From: Undersk0re <44556359+Undersk0re@users.noreply.github.com>
Date: Thu, 2 Jan 2025 15:07:40 +0100
Subject: [PATCH] Update run

fix permisison in /tmp folder to prevent timeout initialization of postgresql like below

"down: postgresql16: 0s, normally up, want up; run: log: (pid 1234) 23s"
---
 srcpkgs/postgresql16/files/postgresql16/run | 1 +
 1 file changed, 1 insertion(+)

diff --git a/srcpkgs/postgresql16/files/postgresql16/run b/srcpkgs/postgresql16/files/postgresql16/run
index 1da79920625df1..5bd4e2d6b7ab3e 100644
--- a/srcpkgs/postgresql16/files/postgresql16/run
+++ b/srcpkgs/postgresql16/files/postgresql16/run
@@ -14,6 +14,7 @@ if [ ! -d "$PGDATA" ]; then
 
 	mkdir -p "$PGDATA" || exit 1
 	chown -R postgres:postgres "$PGDATA"
+	chown -R postgres:postgres /tmp
 	chmod 0700 "$PGDATA"
 	su - postgres -c "/@PREFIX@/bin/initdb $INITOPTS -D '$PGDATA'" 2>&1 || {
 		rm -fr "$PGDATA"

Re: [PR PATCH] [Closed]: Update run permission diring db creation

[Johnnynator]

[-- Attachment #1: Type: text/plain, Size: 689 bytes --]

There's a closed pull request on the void-packages repository

Update run permission diring db creation
https://github.com/void-linux/void-packages/pull/53809

Description:
Fix permissions in `/tmp` folder to prevent initialization timeout for PostgreSQL

This change resolves an issue where PostgreSQL initialization fails with the following error:
"down: postgresql16: 0s, normally up, want up; run: log: (pid 1234) 23s"

# Testing the changes
####    Was this tested? YES
-  This is a minor fix, and it has been verified to address the issue.
-  Security note: Permissions in `/tmp` can be sensitive; please review this commit to ensure it complies with your security policies.

Re: Update run permission diring db creation

[Johnnynator]

[-- Attachment #1: Type: text/plain, Size: 596 bytes --]

New comment by Johnnynator on void-packages repository

https://github.com/void-linux/void-packages/pull/53809#issuecomment-2567854665

Comment:
NO, `chown`-ing `/tmp` to any user is never the correct solution to anything.

> This change resolves an issue where PostgreSQL initialization fails with the following error:
>  "down: postgresql16: 0s, normally up, want up; run: log: (pid 1234) 23s"

This is not an error message, that is just the state. Your system log should have more information.
Feel free to open an issue, if you have logs or a better description of what is going wrong.