search

Ejdamm / market-board

A lighweight marketplace written with PHP Slim framework
MIT License
1 stars 0 forks source link

Add logging to the container #18

Closed Ejdamm closed 5 years ago

Ejdamm commented 5 years ago

/logs/ has 777 permission bits /logs/app.log is created by Monolog and user is set by the user who first runs it. If it is first run by the test-suite it will have user who ran the test as the owner. If it is run ny apache, it will have www-data as user. One user can't append to another user's file unless the user permission is changed for the log files. Shouldn't be a problem in production or test server.

Ejdamm commented 5 years ago

Hm, undra om man kan få in testning här på något sätt. Så att den validerar att loggning skriver dit den ska och att loggningen "innehåller" något vettigt.

Ja jag tänker med det men där är en av mina svagheter med testning. Vilken typ av test handlar det om?