[NETCONF-322] Restfonf returns Status.Ok if delete fails Created: 25/Nov/16  Updated: 15/Mar/19  Resolved: 29/Nov/16

Status: Resolved
Project: netconf
Component/s: restconf-nb
Affects Version/s: None
Fix Version/s: None

Type: Bug
Reporter: Marek Gradzki Assignee: Jakub Toth
Resolution: Done Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified
Environment:

Operating System: All
Platform: All


External issue ID: 7240

 Description   

RestconfImpl.deleteConfigurationData@973 (1.4.0-Boron)

Futures.addListener (used by addCallback) says:

// ListenableFuture's contract is that it will not throw unchecked
// exceptions, so log the bad runnable and/or executor and swallow it.
log.log(Level.SEVERE, "RuntimeException while executing runnable "
+ listener + " with executor " + executor, e);

but RestconfImpl does throw RestconfDocumentedException (RuntimeException)

in handlerLoggerPost.

which results in deleteConfigurationData returning Status.OK.



 Comments   
Comment by Jakub Toth [ 25/Nov/16 ]

https://git.opendaylight.org/gerrit/#/c/48705/

Generated at Wed Feb 07 20:14:43 UTC 2024 using Jira 8.20.10#820010-sha1:ace47f9899e9ee25d7157d59aa17ab06aee30d3d.