Project

General

Profile

Task #3131

Bug #3492: Invalid PIDs in production (whitespace)

Whitespace in pisco PID

Added by Dave Vieglais over 9 years ago. Updated about 7 years ago.

Status:
Closed
Priority:
Low
Assignee:
Ben Leinfelder
Category:
Metacat
Target version:
Start date:
2014-02-12
Due date:
% Done:

100%

Estimated time:
0.00 h
Milestone:
None
Product Version:
Story Points:
Sprint:

Description

There's a single PID on PISCO (perhaps more) that contains whitespace:

doi:10.6085/AA/ SHLX00_XXXITV2XLSR03_20111128.40.1

which suggests that there may be a way to add PIDs to Metacat that circumvents PID conformance checking?

History

#1 Updated by Ben Leinfelder over 9 years ago

  • Status changed from New to In Progress

There was a way to create objects with whitespace in them (https://redmine.dataone.org/issues/3047) that has since been fixed.

But this whitespace id didn't go through the MN.create() method, it was generated from an old Metacat docid that probably has a leading space. The CN Metacat will now reject this pid on CN.create() (during synchronization), but that fix (Metacat 2.0.3) was only just deployed on the CNs three days ago.

Should we attempt to fix this Identifier before it gets replicated elsewhere?

#2 Updated by Ben Leinfelder over 9 years ago

There are only 2 white-space-containing PIDs in the KNB (which includes the PISCO pid originally cited in this bug).

" SHLX00_XXXITV2XLSR03_20111128.40.1"
"doi:10.5063/AA/Virginia Perez.2.1"

#3 Updated by Dave Vieglais over 9 years ago

These should be corrected, just to keep everything consistent and to reduce the chance of unexpected behavior at some point in the future.

#4 Updated by Matthew Jones over 9 years ago

I agree they should be corrected; that will take negotiation with the content owner, which shouldn't pose a problem. -- Matt

#5 Updated by Skye Roseboom over 8 years ago

  • Parent task set to #3492
  • Start date deleted (2012-08-13)
  • translation missing: en.field_remaining_hours set to 0.0

#6 Updated by Robert Waltz about 8 years ago

  • Target version set to 2013.44-Block.6.1

#7 Updated by Chris Jones almost 8 years ago

  • Target version changed from 2013.44-Block.6.1 to 2014.4-Block.1.2

#8 Updated by Jing Tao almost 8 years ago

  • Estimated time set to 0.00
  • Start date set to 2014-02-12
  • Status changed from In Progress to Closed

We added the code in metacat to enforce there is no white spaces in the docid.

#9 Updated by Chris Jones over 7 years ago

  • Target version changed from 2014.4-Block.1.2 to 2014.12-Block.2.2

#10 Updated by Robert Waltz over 7 years ago

  • Target version changed from 2014.12-Block.2.2 to Release Backlog

#11 Updated by Dave Vieglais about 7 years ago

  • Target version changed from Release Backlog to Maintenance Backlog

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 14.8 MB)