Bug #1308
CRUD service interfaces do not allow InvalidRequest for create or update
Start date:
Due date:
% Done:
100%
Milestone:
Product Version:
*
Story Points:
Sprint:
Description
This is a mistake, because it is very possible for the MMP to be malformed by the client. For a relevant example, recall the sysmeta vs. systemmetadata confusion for the name of the part containing the systemMetadata.
Currently GMN does throw an informative InvalidRequest exception, which was quite helpful for debugging the client code quickly.
Metacat follows the spec and throws a 500 error (ServiceFailure), but that masks the problem. (It's error description is "Error handling MMP upload: null", a bit less informative)
Recommend adding the InvalidRequest to the throws clause for:
1. CoordinatingNodeCrud.create
2. MemberNodeCrud.create
3. MemberNodeCrud.update
Subtasks
History
#1 Updated by Matthew Jones over 13 years ago
- Target version set to Sprint-2011.14-Block.2
- Position set to 33
#2 Updated by Dave Vieglais over 13 years ago
- Position deleted (
36) - Position set to 2
- Target version changed from Sprint-2011.14-Block.2 to Sprint-2011.15-Block.2
#3 Updated by Dave Vieglais over 13 years ago
- Position set to 4
- Position deleted (
2)
#4 Updated by Dave Vieglais over 13 years ago
- Position set to 1
- Target version changed from Sprint-2011.15-Block.2 to Sprint-2011.17-Block.3
- Position deleted (
16)
#5 Updated by Rob Nahf over 13 years ago
- Status changed from New to Closed
