As its hard to reliably check how many review apps there are on the
clients machine, we start by checking where the type is `review`. This
means the folder is called that way. This will lead to a seq
scan on the table. However, this is done once a week, so the benefit of
adding an index seems not to apply here.

Fix filename method of GitlabUploader to return always real filename

Closes #33524

See merge request !12113

Ignore CVE-2017-5029 in Nokogiri

Closes #33597

See merge request !12109

Let PhantomJS load local images

See merge request !12003

Set artifact working directory to be in the destination store to prevent unnecessary I/O

Closes #33274

See merge request !11905

Add an optional performance bar to view performance metrics for the current page

Closes #29010

See merge request !11439

Prevent Description Change Notes When Toggling Tasks

Closes #32834

See merge request !12057

This change fix a memory leak due to a Webkit bug:
https://github.com/ariya/phantomjs/issues/12903

Also:

- Whitelist only localhost and 127.0.0.1 in Capybara + JS specs
- Blacklist all requests to media such as images, videos, PDFs, CSVs etc.
- Log all the requests made.

Signed-off-by: Rémy Coutable <remy@rymai.me>