search

rafguns / doidownloader

You give it DOIs, it gives you the article PDFs
MIT License
0 stars 0 forks source link

Investigate: keep PK in doi_fulltext? #15

Closed rafguns closed 1 year ago

rafguns commented 1 year ago

I was about to remove it since we ignore errors anyway. But now that we no longer use MIME types, the text/xml-application/xml situation can no longer happen (I think). So perhaps we should keep it after all?

rafguns commented 1 year ago

I think we should no longer let the IntegrityError pass silently and keep the primary key.

rafguns commented 1 year ago

On branch 'restructure', we log an error if this takes place. This is probably the best solution. In principle, it shouldn't happen but at least we can know that it happens now.