Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Debian package chmod 777 #2659

Closed
piramix opened this issue Jun 28, 2017 · 4 comments
Closed

Debian package chmod 777 #2659

piramix opened this issue Jun 28, 2017 · 4 comments
Assignees

Comments

@piramix
Copy link

piramix commented Jun 28, 2017

Do I really need chmod 777 for all files in this directory /usr/local/kong ? After upgrade every file there is writeable .conf .sh etc.

It's done by debian package postinstall.
#!/bin/sh
mkdir -p /etc/kong
mv /usr/local/lib/luarocks/rocks/kong/0.10.1-0/kong.conf.default /etc/kong/kong.conf.default
chmod -R 777 /usr/local/kong/

@p0pr0ck5
Copy link
Contributor

@thefosk ^

@subnetmarco
Copy link
Member

subnetmarco commented Jul 17, 2017

@p0pr0ck5 the fix for this is creating a user with the appropriate permissions to the default prefix /usr/local/kong and making it easy to configure in Kong (we will probably need some user and group configuration options for nginx).

Since we don't know in advance what user will be running Kong, and what permissions such user has on /usr/local/kong, right now we just take the quick n'dirty path to 777'ing the default Kong prefix.

@thibaultcha
Copy link
Member

(we will probably need some user and group configuration options for nginx).

We have those since #2180

With Nginx packaging this will effectively be a user responsibility via make install.

@p0pr0ck5
Copy link
Contributor

p0pr0ck5 commented Sep 1, 2017

Debian packages no longer chmod 777, starting w/ 0.11.0 release :)

@p0pr0ck5 p0pr0ck5 closed this as completed Sep 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants