Improve CONTRIBUTING guide

Also:

- update the docker-compose to use Alpine image (lighter)
- update the PHP Dockerfile to use Imagick instead of GD (for GIF support when downloading images) and add support for SQLite

Merge pull request #4016 from wallabag/dependabot/composer/nelmio/cors-bundle-1.5.6

Bump nelmio/cors-bundle from 1.5.5 to 1.5.6

Merge pull request #4017 from wallabag/dependabot/composer/friendsofsymfony/jsrouting-bundle-2.3.1

Bump friendsofsymfony/jsrouting-bundle from 2.3.0 to 2.3.1

Bump friendsofsymfony/jsrouting-bundle from 2.3.0 to 2.3.1

Bumps [friendsofsymfony/jsrouting-bundle](https://github.com/FriendsOfSymfony/FOSJsRoutingBundle) from 2.3.0 to 2.3.1.
- [Release notes](https://github.com/FriendsOfSymfony/FOSJsRoutingBundle/releases)
- [Changelog](https://github.com/FriendsOfSymfony/FOSJsRoutingBundle/blob/master/CHANGELOG.md)
- [Commits](https://github.com/FriendsOfSymfony/FOSJsRoutingBundle/compare/2.3.0...2.3.1)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Bump nelmio/cors-bundle from 1.5.5 to 1.5.6

Bumps [nelmio/cors-bundle](https://github.com/nelmio/NelmioCorsBundle) from 1.5.5 to 1.5.6.
- [Release notes](https://github.com/nelmio/NelmioCorsBundle/releases)
- [Changelog](https://github.com/nelmio/NelmioCorsBundle/blob/master/CHANGELOG.md)
- [Commits](https://github.com/nelmio/NelmioCorsBundle/compare/1.5.5...1.5.6)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Merge pull request #4009 from wallabag/fix/check-git-directory

Check if .git directory exist before making the update

Return a different code when git isn't installed

Merge pull request #4010 from wallabag/dependabot/composer/doctrine/doctrine-fixtures-bundle-3.2.2

Bump doctrine/doctrine-fixtures-bundle from 3.2.0 to 3.2.2

Check if .git directory exist

If that folder doesn’t exist, it means the udpate script won’t be able to run because it uses git to retrieve the update.

Bump doctrine/doctrine-fixtures-bundle from 3.2.0 to 3.2.2

Bumps [doctrine/doctrine-fixtures-bundle](https://github.com/doctrine/DoctrineFixturesBundle) from 3.2.0 to 3.2.2.
- [Release notes](https://github.com/doctrine/DoctrineFixturesBundle/releases)
- [Changelog](https://github.com/doctrine/DoctrineFixturesBundle/blob/master/CHANGELOG-3.1.md)
- [Commits](https://github.com/doctrine/DoctrineFixturesBundle/compare/3.2.0...3.2.2)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Merge pull request #4008 from wallabag/travis-xenial

Fix Travis build on Xenial dist

Merge pull request #3995 from wallabag/feature/config-display-login

Display login in config / user

Fix Travis build on Xenial dist

Typo

Update FUNDING.yml

"Some users provided are not enrolled in GitHub Sponsors."

Merge pull request #4006 from wallabag/add-sponsor-badge

Create FUNDING.yml

Create FUNDING.yml

Merge pull request #4004 from wallabag/dependabot/composer/php-http/httplug-bundle-1.16.0

Bump php-http/httplug-bundle from 1.15.2 to 1.16.0

Bump php-http/httplug-bundle from 1.15.2 to 1.16.0

Bumps [php-http/httplug-bundle](https://github.com/php-http/HttplugBundle) from 1.15.2 to 1.16.0.
- [Release notes](https://github.com/php-http/HttplugBundle/releases)
- [Changelog](https://github.com/php-http/HttplugBundle/blob/master/CHANGELOG.md)
- [Commits](https://github.com/php-http/HttplugBundle/compare/1.15.2...1.16.0)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Merge pull request #3999 from wallabag/dependabot/composer/symfony/phpunit-bridge-4.3.1

Bump symfony/phpunit-bridge from 4.3.0 to 4.3.1

Bump symfony/phpunit-bridge from 4.3.0 to 4.3.1

Bumps [symfony/phpunit-bridge](https://github.com/symfony/phpunit-bridge) from 4.3.0 to 4.3.1.
- [Release notes](https://github.com/symfony/phpunit-bridge/releases)
- [Changelog](https://github.com/symfony/phpunit-bridge/blob/master/CHANGELOG.md)
- [Commits](https://github.com/symfony/phpunit-bridge/compare/v4.3.0...v4.3.1)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Merge pull request #3998 from wallabag/dependabot/composer/doctrine/doctrine-fixtures-bundle-3.2.0

Bump doctrine/doctrine-fixtures-bundle from 3.1.0 to 3.2.0

Bump doctrine/doctrine-fixtures-bundle from 3.1.0 to 3.2.0

Bumps [doctrine/doctrine-fixtures-bundle](https://github.com/doctrine/DoctrineFixturesBundle) from 3.1.0 to 3.2.0.
- [Release notes](https://github.com/doctrine/DoctrineFixturesBundle/releases)
- [Changelog](https://github.com/doctrine/DoctrineFixturesBundle/blob/master/CHANGELOG-3.1.md)
- [Commits](https://github.com/doctrine/DoctrineFixturesBundle/compare/3.1.0...3.2.0)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Update translations

Show untagged entries count on tag list (#3993)

Show untagged entries count on tag list

Display login in config / user

Just so the user can see its login

Remove link when there are no untagged articles

Merge pull request #3959 from wallabag/mig-tag-collation

mysql: change collation of tag label

Merge pull request #3994 from wallabag/feature/revoke-feed-token

Add ability to revoke feed token

Add ability to revoke feed token

Merge pull request #3992 from wallabag/feature/menu-quickstart

Add quickstart into the menu

Fix typo & CS

Show untagged entries count on tag list

Closes #3235

Signed-off-by: Kevin Decherf <kevin@kdecherf.com>

Add quickstart into the menu

Just in case people want to see that page again.

Merge pull request #3990 from wallabag/feature/sentry

Add Sentry support

Merge pull request #3991 from wallabag/fix/secret

Try to force people to update the secret

Try to force people to update the secret

Add Sentry support

Only enable in production

Merge pull request #3271 from wallabag/store-resolved-url

Add `given_url` in Entry table to check if a redirected url has already added

Typos

Use two indexes instead of one for hashed urls

When using `OR` in a where clause, a composite index can't be used. We should use a `UNION` to take advantages of it.
Instead, create 2 indexes on each hashed urls and make 2 queries to find an url. It'll be faster than the previous solution.

Merge pull request #3987 from wallabag/fix/drop-simplepie

Drop SimplePie

Drop SimplePie

It was only used to make an absolute url when downloading images.
The deps is still there (in the `composer.lock`) because Graby use it (not for absolute but for encoding).

Use hash given url to avoid duplicate

Using hashed url we can ensure an index on them to ensure it's fast.

Merge pull request #3986 from wallabag/fix/scrutinizer-issues

Fix some Scrutinizer issues

Added given_url in entry table

- Added index on entry table for given_url field
- Fix tests:

    The previous `bit.ly` url redirected to doc.wallabag but that url doesn't exist in the fixtures.
    I used our own internal "redirector" to create a redirect to an url which exist in the fixtures.

Also, updating current migration to use the new `WallabagMigration`.

Fix some Scrutinizer issues

Merge pull request #3985 from wallabag/update-composer

Keep composer.lock

Typos

Keep composer.lock

We initially ignored the `composer.lock` because it generated a lock of rebase on PR when someone updated it and the master updated it too.
Now we have less contributions (sadly) so I think we won't run against that problem.
Also, it'll solve issue about people cloning the master and got angry because composer eat all the available memory to determine packages to install.
It'll also be much easier to make release.
Scrutinizer & Travis will be faster too.

Merge pull request #3984 from wallabag/2.4

Merge 2.4 into master

Merge pull request #3944 from shtrom/always-hash-exists-url

Always hash exists url

Merge pull request #3390 from aaa2000/httplug

Use httplug and graby 2.0

Fix tests after rebase

CS & fix tests

Fix some tests

CS

CS

Configure timeout

Fix because of some breaking changes of Graby 2.0

Use httplug

Merge remote-tracking branch 'origin/master' into 2.4

Merge pull request #3982 from wallabag/fix/https-test

Fix http/https test

Fix test

Merge pull request #3965 from nicofrand/previewPic

Preview picture: use the 1st pic retrieved if no og:image set

Merge pull request #3980 from wallabag/fix/instapaper-date-import

Fix Instapaper import date order

Remove useless methods

Also fix a phpdoc block

Change the way to define algorithm for hashing url

Add Wallabag\CoreBundle\Helper\UrlHasher

Signed-off-by: Olivier Mehani <shtrom@ssji.net>

Delegate findByUrlAndUserId to findByHashedUrlAndUserId

Signed-off-by: Olivier Mehani <shtrom@ssji.net>

EntryRestController::getEntriesExistsAction: always find by hashed url

Simplify the logic from #3158 by hashing all the urls from the request,
and only doing a search by hash. This allows to get performance benefits
from the new indexed hash column even when using older clients that do
not hash the URL in the request.

Fixes: #3158, #3919
Signed-off-by: Olivier Mehani <shtrom@ssji.net>

Fix Instapaper import date

Merge pull request #3977 from wallabag/fix/7.2-dockerfile

Force PHP version in Dockerfile

Force PHP version in Dockerfile

[tests] Fix pre-existing tests (preview now imported + records added)

[tests] Set first picture as preview picture

Set first picture as preview picture

Merge pull request #3960 from wallabag/api-entries-2817

api/entries: add parameter detail to exclude or include content in response

mysql: change collation of tag table

utf8mb4_unicode_ci considers that 'caché' is equal to 'cache' which
can lead to attaching incorrect tags to entries. This issue is due to
some unicode normalization done by MySQL.

utf8mb4_bin makes no unicode normalization, letting wallabag to consider
'cache' and 'caché' as two different tags.

We change the collation of the whole table as Doctrine does not support
setting a collation on a column for a specific platform (it tries to
apply utf8mb4_bin even for pgsql and sqlite).

Fixes #3302

Signed-off-by: Kevin Decherf <kevin@kdecherf.com>

api/entries: add parameter detail to exclude or include content in response

detail=metadata will nullify the content field of entries in order to
make smaller responses.

detail=full keeps the former behavior, it sends the content of entries.
It's the default, for backward compatibility.

Fixes #2817

Signed-off-by: Kevin Decherf <kevin@kdecherf.com>

CS

Merge remote-tracking branch 'origin/master' into 2.4

Merge pull request #3966 from wallabag/prepare-2.3.8

Prepare 2.3.8 release

Prepare 2.3.8 release

Merge pull request #3964 from wallabag/fix/intl

Fix Intl Locale issue

Fix Intl Locale issue

Some issue appeared after the release of PHP 7.2.17 about Intl Memory
Leak / infinite loop.
To fix it we should upgrade to Symfony 3.4 (which is done in wallabag
2.4) but for the 2.3 branch, we'll use a temporary fix for the Locale
issue.

Merge pull request #3956 from wallabag/fix/delete-cascade-oauth2

Cascade delete on oauth2 table when deleting a user

Merge pull request #3922 from burkemw3/patch-5

Remove preview picture from share view page for #1875

Merge pull request #3943 from wallabag/img-referrer

Enable no-referrer on img tags, enable strict-origin-when-cross-origin by default

Remove preview picture from share view page for #1875

Essentially, same as commit 038fccd for single entry views. From that commit:
> Showing the preview picture usually leads to showing a duplicate
> image, and frequently leads to showing duplicate images directly
> adjacent to each other.

assets: regenerate material.css

Signed-off-by: Kevin Decherf <kevin@kdecherf.com>

Enable no-referrer on img tags, enable strict-origin-when-cross-origin by default

Fixes #3889

Signed-off-by: Kevin Decherf <kevin@kdecherf.com>

Fix SQLite constraint

Fix SQLite migration

Cascade delete on oauth2 table when deleting a user

Merge pull request #3955 from wallabag/fix/gif-animation-imagick

Use Imagick to keep GIF animation

Fallback to default solution if Imagick fails

CS

Setup Imagick for Travis

To avoid error from phpstan about class not found

Use Imagick to keep GIF animation

If Imagick is available, GIF will be saved using it to keep animation.
Otherwise the previous method will be used and the animation won't be kept.