wallabag / wallabagger

Chrome / Firefox / Opera plugin for wallabag v2.

http://wallabag.github.io/wallabagger/

MIT License

363 stars 48 forks source link

Content fetching from browser is missing author and pub date #387

Open johnlago opened 4 months ago

johnlago commented 4 months ago

When using the "Content fetching from the browser" option, Wallabag/Wallabagger seems to consistently not grab an article's author and publication date. Having the Wallabag server fetch the article itself, those fields do fill in.

This article can be used to see the difference: https://www.theverge.com/24064103/video-game-history-foundation-digital-library

HolgerAusB commented 4 months ago

I am not sure, if the problem is on wallabag or on wallabagger. Please also see https://github.com/wallabag/wallabag/issues/6972

charleshan commented 3 months ago

Wallabagger is using Wallabag's APIs so my guess is that it's a problem with Wallabagger.

What steps will reproduce the bug?

Wallabagger > Options > Content fetching from the browser > Enable

When "Content fetching from the browser" is enabled, the author is blank. When "Content fetching from the browser" is disabled, the author is set.

From my experience, it's always better to use "Content fetching from the browser" so I hope we can get this fixed

charleshan commented 3 months ago

After digging a bit deeper, it's possible that it's a problem with Wallabag...

We should first test if the title is actually set here: https://github.com/wallabag/wallabagger/blob/3c811e40d555b0cfceffe831728029a3c889066a/wallabagger/js/background.js#L502C1-L505C6

And then check how Wallabag handles the POST request.