search

ministryofjustice / cla_frontend

CLA Front End
http://ministryofjustice.github.io/cla_docs/
MIT License
3 stars 4 forks source link

Remove mod-security #819

Closed said-moj closed 2 years ago

said-moj commented 2 years ago

What does this pull request do?

Remove mod-security

Any other changes that would benefit highlighting?

With mod-security enabled we are experiencing a noticeable difference in requests performance which resulted in us increasing request timeout parameters and provider csvs not upload.

For example with mod-security enabled /provider/csvupload/ POST request with a 99kb file was taking upto 3 minutes to return feedback to the user. With mod-security removed the same request now takes around 700ms at most.

We need to investigate why this happens when mod-security is enabled however at the moment the performance of the frontend has degraded to a point where it is noticeable to the users

Checklist

said-moj commented 2 years ago

Closing in favour of less drastic solution to remove json parsing with mod-security instead https://github.com/ministryofjustice/cla_frontend/pull/820