Benoît Viguier comments

Results 246 comments of


Benoît Viguier

Falsely claims: X-Content-Type-Options was not set to "nosniff"

Another approach would be white listing some domains which are known to publish static pages by trusted processes.

Full source code in the docker image

Then why is this not working? https://github.com/LycheeOrg/Lychee-Docker/blob/d10de82598ff3b696aa3a5b47fa17cd02b2b46d7/Dockerfile#L65-L66

Full source code in the docker image

When running it on my instance, it does find the, ```sh root@vps:/var/docker/Lychee-docker perso ✗ 167d14h ◒ ▶ docker exec -it lychee bash root@e7d513f87a2f:/var/www/html/Lychee# find . -wholename '*/[Tt]ests/*' ./tests/LoadedSubscriber.php ./tests/Samples ./tests/Samples/xcf.xcf...

Full source code in the docker image

AFAIK `.dockerignore` won't help in this case. We are cloning inside the image, not outside and then doing a copy.

Full source code in the docker image

Nicely spotted!

Change album cover picture creates a slide show of all the picture after the selected one

It is more of a bug of the layout. It is in my todo fix.

Change album cover picture creates a slide show of all the picture after the selected one

Seems like I will need to do a bit of refactoring on the menus...

Refactor image processing to use pipeline instead of hard to read strategy

@kamil4 opinion ? @d7415 ? :)

Refactor image processing to use pipeline instead of hard to read strategy

@Kovah heads up. But I think this should make it easier for you to understand the add photo code flow.

Refactor image processing to use pipeline instead of hard to read strategy

> Would have been nice to know this earlier. Spent a few hours trying to get into the structure, and can now spend more time changing everything. Not cool. Yeah...