Bug 2521 - Wrong return status code for non-existing mount-point 66/21866/2
authorVaclav Demcak <vdemcak@cisco.com>
Tue, 2 Jun 2015 13:14:48 +0000 (15:14 +0200)
committerGerrit Code Review <gerrit@opendaylight.org>
Mon, 8 Jun 2015 08:37:31 +0000 (08:37 +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>
(cherry picked from commit 097cf9e9d96e6ad94ab91728ab84c49b0eba5f61)

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();