Merge pull request #3991 from wallabag/fix/secret

Try to force people to update the secret

Try to force people to update the secret

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

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.

Merge pull request #3942 from wallabag/issue-3838

material: add metadata to list view

Merge pull request #3223 from wallabag/rss2atom

Changed RSS to Atom feed and improve paging

material: add metadata to list view

Add reading time and creation date to rows of list view.
Refactor styles using a sass mixin.

Fixes #3838

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

Merge pull request #3937 from wallabag/fix/credential-subdomain

 Add ability to match many domains for credentials

Handle redirection from previous feeds

Fix tests & cs & migration

Changed RSS to Atom feed and improve paging

Merge pull request #3158 from wallabag/hash-exist-url

Hash exist url

Adding more tests to cover different scenario

FIx unrelated failing test

Add ability to match many domains for credentials

Instead of fetching one domain, we use the same method as in site config (to retrieve the matching file) and handle api.example.org, example.org, .org (yes the last one isn’t useful).
If one of these match, we got it and use it.

Fix ApiDoc about md5/sha1

Fix index on MySQL

Some cleanup

Also, do not run the hashed_url migration into a Doctrine migration

Use a better index for hashed_url

It'll most often be used in addition to the `user_id`.
Also, automatically generate the hash when saving the url.
Switch from `md5` to `sha1`.

Keep url in exists endpoint

- Add migration
- Use md5 instead of sha512 (we don't need security here, just a hash)
- Update tests

Hash the urls to check if they exist

Signed-off-by: Thomas Citharel <tcit@tcit.fr>

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

Merge pull request #3909 from wallabag/fix/html-not-defined

Fix PHP warning

Merge pull request #3908 from wallabag/epub-issue-3642

epub: fix exception when articles have the same title

Merge pull request #3860 from baurmatt/fix/posix_compatibility

Make dev/install/update script posix compatible

Fix PHP warning

Looks like sometimes (usually from import) the `html` key isn’t available.

epub: fix exception when articles have the same title

This commit fixes an exception occuring when exporting as epub several
articles with the same title. The chapter filename is now derived from
title and url.

Fixes #3642

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

Properly source require.sh

Fixes #3571.

Merge pull request #3901 from wallabag/fix-pr-3893

material: fix left padding on non-entry pages introduced by #3893

material: fix left padding on non-entry pages introduced by #3893

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

Merge pull request #3897 from wallabag/2.3.8-dev

Jump to 2.3.8-dev

Jump to 2.3.8-dev

Merge pull request #3895 from wallabag/prepare-2.3.7

Prepare 2.3.7 release

Prepare 2.3.7 release

Merge pull request #3894 from wallabag/issue-3851

material: hide creation date from card actions on specific sizes

material: update prod assets

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

material: add media queries to hide creation date from card actions

Hide the creation date from card actions on specific sizes when there's
not enough space for all parts.

Fixes #3851

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

material: wrap card actions, remove class hiding of creation date

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

Merge pull request #3893 from wallabag/issue-3877

material: fix left padding of content on medium screens

material: fix left padding of content on medium screens

Fixes #3877

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

material: move a media query from cards to dedicated scss file

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