engineyard / engineyard/eycap

Misleading comments and use of sudo method in mongrel.rb recipes

Open
#2 3 comments 0 reactions 0 assignees View on GitHub
Dominant language
Ruby
Stars
115
Forks
55
PR merge metrics
No merged PRs in 30d

Description

The tasks in the mongrel namespace are using the sudo method to interact with monit, but the comments allude to the fact that they can be run without sudo access by toggling :use_sudo. This is not true, however, because the sudo command _always_ runs using sudo. The commands should most likely use the _try_sudo_ command instead, which looks at the :use_sudo flag or if :run_method is :sudo.

This was reported by jonah_ in IRC.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.