Project

General

Profile

Task #3131

Bug #3492: Invalid PIDs in production (whitespace)

Whitespace in pisco PID

Added by Dave Vieglais over 11 years ago. Updated over 9 years ago.

Status:
Closed
Priority:
Low
Assignee:
Ben Leinfelder
Category:
Metacat
Target version:
Maintenance Backlog
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 11 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 11 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 11 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 11 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 almost 11 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 over 10 years ago

  • Target version set to 2013.44-Block.6.1

#7 Updated by Chris Jones over 10 years ago

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

#8 Updated by Jing Tao about 10 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 about 10 years ago

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

#10 Updated by Robert Waltz over 9 years ago

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

#11 Updated by Dave Vieglais over 9 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)