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

Make RPM so that nginx is supported too #143

Closed
finkelamein opened this Issue Oct 29, 2014 · 3 comments

Comments

Projects
None yet
2 participants
@finkelamein

finkelamein commented Oct 29, 2014

For the rpm specfile: If lighthttpd and/or maybe nginx are supported as well to serve Seccubus, maybe a RPM requirement "webserver" instead of "httpd" would be more appropriate?

@seccubus

This comment has been minimized.

Member

seccubus commented Oct 29, 2014

@finkelamein the .conf file that is dropped by Seccubus is specific to apache.
Do you know if it is possible to test for the webserver installed in a .spec file and execute the appropriate actions?

@finkelamein

This comment has been minimized.

finkelamein commented Oct 30, 2014

https://docs.fedoraproject.org/en-US/Fedora_Draft_Documentation/0.1/html/RPM_Guide/ch09s05s03.html gives some sort of hint... although it's draft documentation
%config(noreplace) /etc/yp.conf

Wat that would entail is probably place all options in the rpm "nginx, httpd, lighthttpd" and remove the once that are not needed in postscript. or just leave httpd in there and remove when httpd is not installed.

@seccubus seccubus added this to the Minor feature requests milestone Jun 7, 2016

@seccubus seccubus changed the title from RPM requirements to Make RPM so that nginx is supported too Jun 7, 2016

@seccubus

This comment has been minimized.

Member

seccubus commented Jun 15, 2017

With PR #411 Seccubus no longer depends on a webserver but is Using Mojolicious instead.

@seccubus seccubus closed this Jun 15, 2017

seccubus added a commit that referenced this issue Jun 15, 2017

@seccubus seccubus referenced this issue Jun 15, 2017

Merged

Release v2.34 #467

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment