akhilrex / podgrab

A self-hosted podcast manager/downloader/archiver tool to download podcast episodes as soon as they become live with an integrated player.
GNU General Public License v3.0
1.55k stars 88 forks source link

Possible Bug - Environemnt variable PID #197

Open moorgrove opened 2 years ago

moorgrove commented 2 years ago

When looking at the environment variables i found that PID is set to 100 but i cant find any reference to PID in the code. But i can find that variable GID is used in service/fileService.go line 177 and 181.

uid, err1 := strconv.Atoi(os.Getenv("PUID"))
gid, err2 := strconv.Atoi(os.Getenv("PGID"))

I suspect that the environment variables UID and PID should be renamed to PUID and PGID?

akhilrex commented 2 years ago

Let me look into this.

On Thu, 24 Feb 2022 at 21:07, Andreas Hedlund @.***> wrote:

When looking at the environment variables i found that PID is set to 100 but i cant find any reference to PID in the code. But i can find that variable GID is used in service/fileService.go https://github.com/akhilrex/podgrab/blob/8a2f7af9e85efc1c7729c41003a060c08fbee07a/service/fileService.go line 177 and 181.

uid, err1 := strconv.Atoi(os.Getenv("PUID"))

gid, err2 := strconv.Atoi(os.Getenv("PGID"))

´´´

I suspect that the environment variables UID and PID should be renamed to PUID and PGID?

— Reply to this email directly, view it on GitHub https://github.com/akhilrex/podgrab/issues/197, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAEMVPQHL74HXQWLBAQHKMDU4ZGDLANCNFSM5PHXFV5Q . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you are subscribed to this thread.Message ID: @.***>

-- Thanks and Regards, Akhil Gupta