search

inspirehep / inspire-next

The INSPIRE repo.
https://inspirehep.net
GNU General Public License v3.0
59 stars 69 forks source link

Bug in reporting ticket(?) #2348

Closed kaplun closed 7 years ago

kaplun commented 7 years ago

See Sentry entry: https://sentry.cern.ch/inspire-sentry/inspire-labs/group/822209/ Blocked record is: https://labs.inspirehep.net/holdingpen/639610

kaplun commented 7 years ago

Looks like the obj['_extra_data'] was not having a reason key at the time @annetteholtkamp clicked. Now it contains an empty string: https://labs.inspirehep.net/api/holdingpen/639610

The source code is doing obj['_extra_data'].get('reason').strip() which of course might crash

jacquerie commented 7 years ago

Yup, as I said this morning: https://github.com/inspirehep/inspire-next/pull/2313#issuecomment-302305699 (or 11 days ago...)