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

Use wsgi server for production #53

Open
fritterhoff opened this issue Jul 15, 2021 · 4 comments
Open

Use wsgi server for production #53

fritterhoff opened this issue Jul 15, 2021 · 4 comments
Assignees
Labels
Discuss Topics to discuss enhancement New feature or request Extractor Extractor related issue Sysmon Sysmon related issue
Milestone

Comments

@fritterhoff
Copy link
Collaborator

At the moment the default flask server is used for production. It is best practice/recommended to use an wsgi server like Gunicorn for production.
As far as I know, the scheduler could cause some issues due to the fork modell of the server (depending on the individual solution).

@DerAlexmeister
Copy link
Collaborator

DerAlexmeister commented Jul 15, 2021

Does this solve the Problem mentioned in #24. Only the Extractor and the Sysmon should be exposed. @fritterhoff

@DerAlexmeister DerAlexmeister added Discuss Topics to discuss enhancement New feature or request labels Jul 15, 2021
@DerAlexmeister DerAlexmeister added this to the 0.1.2 milestone Jul 15, 2021
@DerAlexmeister DerAlexmeister added Extractor Extractor related issue Sysmon Sysmon related issue labels Jul 15, 2021
@fritterhoff
Copy link
Collaborator Author

Does this solve the Problem mentioned in #24. @fritterhoff

Maybe it makes integrating the reverse proxy easier.

@fritterhoff
Copy link
Collaborator Author

fritterhoff commented Jul 15, 2021

In detail the problem comes from:

<link rel="stylesheet" href="/static/bootstrap/css/bootstrap.min.css">

Here you reference the static resources directly without any prefix. We should at least change that to

<link rel="stylesheet" href="{{ url_for('static', filename='bootstrap/css/bootstrap.min.css') }}">

@fritterhoff
Copy link
Collaborator Author

According to https://dlukes.github.io/flask-wsgi-url-prefix.html that should solve the problem in combination with the env var SCRIPT_NAME.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Discuss Topics to discuss enhancement New feature or request Extractor Extractor related issue Sysmon Sysmon related issue
Projects
None yet
Development

No branches or pull requests

2 participants