Project

General

Profile

Bug #1076

mn.create() not encoding PIDs correctly.

Added by Rob Nahf over 13 years ago. Updated over 13 years ago.

Status:
Closed
Priority:
Urgent
Assignee:
Category:
d1_libclient_java
Target version:
Start date:
2010-11-13
Due date:
% Done:

100%

Estimated time:
(Total: 1.00 h)
Milestone:
Product Version:
*
Story Points:
Sprint:

Description

For cn/resolve testing, I try creating objects with identifiers containing different types of characters that need to be escaped to be a path segment in a url (for the rest calls). All escaping tests fail (disallowed ascii and non-ascii in the 1,2,3,4 byte varieties).
Looking into the code, it looks like encoding gets mangled in D1Node.createMimeMultipart().


Subtasks

Task #1077: pct-escaping in D1Node.java or MNode.javaClosedRob Nahf

Task #1078: encoding function needed for identifiers in the param section of urlsClosedRob Nahf

Task #1125: mn.create() returning 400 error for escaped identifiersClosedChad Berkley

Task #1153: Include identifier test set in GMN testsRejectedRoger Dahl


Related issues

Blocked by Infrastructure - Bug #1107: Hudson has dependency problem in d1_libclient_java Closed

History

#1 Updated by Rob Nahf over 13 years ago

  • Position set to 1

#2 Updated by Rob Nahf over 13 years ago

  • Status changed from New to In Progress
  • Assignee set to Rob Nahf

#3 Updated by Dave Vieglais over 13 years ago

  • Position set to 31
  • Target version changed from Sprint-2010.46 to Sprint-2010.47
  • Position deleted (21)

#4 Updated by Rob Nahf over 13 years ago

  • Position deleted (29)
  • Position set to 2
  • Target version changed from Sprint-2010.47 to Sprint-2010.48

#5 Updated by Rob Nahf over 13 years ago

Still getting errors on create after a series of upgrades to libclient error handling by Chad. The return from the http request is an error 400 - bad request. Need to look at how metacat handles unicode.

#6 Updated by Rob Nahf over 13 years ago

  • Status changed from In Progress to Closed

#7 Updated by Rob Nahf over 13 years ago

  • Status changed from Closed to In Progress

#8 Updated by Dave Vieglais over 13 years ago

  • Target version changed from Sprint-2010.48 to Sprint-2010.50
  • Position deleted (36)
  • Position set to 5

#9 Updated by Dave Vieglais over 13 years ago

  • Target version changed from Sprint-2010.50 to Sprint-2011.01

#10 Updated by Rob Nahf over 13 years ago

  • Subject changed from mn.create() not encoding identifiers (guids) correctly. to mn.create() not encoding PIDs correctly.

#11 Updated by Rob Nahf over 13 years ago

  • Status changed from In Progress to Closed

integration tests D1ClientTests work against PIDs with ascii characters. Unicode characters will not work, and issue is deferred.

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 14.8 MB)