search

FAForever / faf-stack

Aggregate project that allows to set up and manage a complete FAF environment within minutes
16 stars 18 forks source link

Remove php completely from content server #149

Closed Brutus5000 closed 4 years ago

Brutus5000 commented 4 years ago

As the name indicates content server is supposed to be for file serving only.

Right now it also needs to support various php files for the vault in the python client. This is an unnecessary security risk and also prevents us from using optimized software / docker images.

Python client alternative 1: Use the java api endpoints as available since Q1 2018. Python client alternative 2: Integrate a new web based vault (work in progress on https://brutus5000.github.io/faf-ng-client)

Brutus5000 commented 4 years ago

This was performed today. The actual reason was huge performance issues with the apache php 5.6 image due to a hotfix patch during prime time.

Brutus5000 commented 4 years ago

The php files have also been removed from the server and archived.