Bug 2521 - Wrong return status code for non-existing mount-point 62/21662/2
authorVaclav Demcak <vdemcak@cisco.com>
Tue, 2 Jun 2015 13:14:48 +0000 (15:14 +0200)
committerGerrit Code Review <gerrit@opendaylight.org>
Thu, 4 Jun 2015 11:00:40 +0000 (11:00 +0000)
Note: MountpointService returns Optional object without mountpoint,
so this is the point where we would like to see error code 404.

Change-Id: Id7902e5c80786275c82e279ae0433a011fa486fc
Signed-off-by: Vaclav Demcak <vdemcak@cisco.com>
opendaylight/md-sal/sal-rest-connector/src/main/java/org/opendaylight/controller/sal/restconf/impl/ControllerContext.java

index 8b5951197d93deb9554e3c9b7f698be1ed06d6af..ad2409bfcf13be60407f853cdce454abf49c1d09 100644 (file)
@@ -536,7 +536,7 @@ public class ControllerContext implements SchemaContextListener {
                 if (!mountOpt.isPresent()) {
                     LOG.debug("Instance identifier to missing mount point: {}", partialPath);
                     throw new RestconfDocumentedException("Mount point does not exist.", ErrorType.PROTOCOL,
-                            ErrorTag.UNKNOWN_ELEMENT);
+                            ErrorTag.DATA_MISSING);
                 }
                 final DOMMountPoint mount = mountOpt.get();