Skip to content
New issue

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

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

Already on GitHub? # to your account

new puma user log #122

Merged
merged 3 commits into from
Aug 19, 2015
Merged

new puma user log #122

merged 3 commits into from
Aug 19, 2015

Conversation

seuros
Copy link
Owner

@seuros seuros commented Aug 19, 2015

@mcb
i refactored the code so we can use the server proprieties instead .
this way we can have different user for each server.

server 'server1', user: 'superuser', roles: %w(web app db), run_as: 'simple_user'
server 'server2', user: 'superuser2', roles: %w(web app), puma_user: 'puma_user'

@mcb
Copy link
Contributor

mcb commented Aug 19, 2015

@seuros Looks good to me 👍

seuros added a commit that referenced this pull request Aug 19, 2015
@seuros seuros merged commit 25d1801 into master Aug 19, 2015
@seuros seuros deleted the puma_user branch August 19, 2015 11:21
@seuros
Copy link
Owner Author

seuros commented Aug 19, 2015

@mcb released. Thank you for your work,

@mcb
Copy link
Contributor

mcb commented Aug 19, 2015

@seuros Thanks for merging. This helps quite a lot.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants