[Bug 214830] Review Request: Limph - PHP network host monitor
bugzilla at redhat.com
bugzilla at redhat.com
Thu Nov 16 21:21:13 UTC 2006
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: Limph - PHP network host monitor
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214830
orion at cora.nwra.com changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |ASSIGNED
AssignedTo|nobody at fedoraproject.org |orion at cora.nwra.com
OtherBugsDependingO|163776, 177841 |163778
nThis| |
------- Additional Comments From orion at cora.nwra.com 2006-11-16 16:21 EST -------
Initial comments:
- BAD: md5sum does not match upstream.
- Need full URL for source (http://prdownloads.sf.net/limph/limph-%{version}.tar.gz
- Please, BuildArch: not BuildArchitectures:
$ rpmlint ~/Desktop/limph-1.9.3-1.src.rpm
W: limph strange-permission limph-hostagent 0755
- perhaps just change the permission with "install"?
E: limph use-of-RPM_SOURCE_DIR
- Instead of "$RPM_SOURCE_DIR/limph.conf", use %SOURCE1, etc.
- should add to limph.conf:
<Directory /usr/share/limph/>
Order Deny,Allow
Deny from all
Allow from 127.0.0.1
</Directory>
- More rpmlint:
# rpmlint limph
E: limph file-in-usr-marked-as-conffile /usr/share/limph/config.php
E: limph file-in-usr-marked-as-conffile /usr/share/limph/input.php
- move to /etc/limph. Is input.php really a config file?
E: limph world-writable /usr/share/limph/tmp 0777
E: limph non-standard-dir-perm /usr/share/limph/tmp 0777
- These should only be writable by limph.
# rpmlint limph-hostagent
E: limph-hostagent file-in-usr-marked-as-conffile /usr/share/limph/host_agent.php
- same here. Maybe put $secret into a config file in /etc/limph and include that?
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.
More information about the Fedora-package-review
mailing list