Maniphest T4275

CffMapping: Add checks for value types
Closed, MigratedEdits Locked
Actions

Assigned To

gitlab-migration

Authored By

	vlorentz
	May 24 2022, 5:37 PM

Tags

Subscribers

gitlab-migration

Description

For example, the crash in https://sentry.softwareheritage.org/share/issue/a67ff03ed63448d0bf386f832564f380/

is caused by @id being None because of https://archive.softwareheritage.org/swh:1:cnt:b7d98a32465e15415619d9b40ed6c58e218d78ad;origin=https://github.com/int-brain-lab/ibllib;visit=swh:1:snp:9027c747eafbdeb342f2c0a32b9374a850014567;anchor=swh:1:rev:e4a4f8895efb2d3a0ae5097ee619b1629ee6c537;path=/CITATION.cff;lines=6

Revisions and Commits

rDCIDX Metadata indexer
	D8036	rDCIDX1be4e184d4b9 cff: Add checks for value types

Related Objects
Search...

		Status	Assigned	Task
		Migrated	gitlab-migration	T3097 Expose metadata in the WebApp and make it searchable
		Migrated	gitlab-migration	T2064 Add metadata from deposits to metadata search
		Migrated	gitlab-migration	T2073 Index extrinsic metadata from the journal in swh-search/Elasticsearch
		Migrated	gitlab-migration	T4273 Rewrite indexers as journal clients when relevant
		Migrated	gitlab-migration	T4282 Deploy new origin intrinsic metadata journal client indexer > v1.1
		Migrated	gitlab-migration	T4274 Resolve all known crashes in the metadata indexer
		Migrated	gitlab-migration	T4275 CffMapping: Add checks for value types
				Restricted Maniphest Task
		Migrated	gitlab-migration	T4401 Index metadata from the deposit
		Migrated	gitlab-migration	T4459 Deploy swh-indexer > v2.6 on staging then production
		Migrated	gitlab-migration	T4429 Deploy swh-indexer v2.3.0 on production and staging
		Migrated	gitlab-migration	T4477 staging origin intrinsic metadata indexer are stuck
		Migrated	gitlab-migration	T4606 Deploy swh-indexer v2.7.0
		Migrated	gitlab-migration	T4694 Use directory metadata in origin search

Event Timeline

vlorentz triaged this task as Normal priority.May 24 2022, 5:37 PM

vlorentz created this task.

vlorentz added a project: Easy hack.

vlorentz removed vlorentz as the assignee of this task.May 24 2022, 5:43 PM

vlorentz assigned this task to VickyMerzOwn.Jun 21 2022, 1:32 PM

I'm not so sure what needs to be done here.
If some fields have no values, does it have to skip it or put some default value?

VickyMerzOwn added a revision: D8036: Check CFF Value Types.Jun 24 2022, 8:01 PM

VickyMerzOwn added a commit: rDCIDX1be4e184d4b9: cff: Add checks for value types.Jun 29 2022, 11:01 AM

VickyMerzOwn closed this task as Resolved.Jun 29 2022, 11:15 AM

This task has been migrated to GitLab.