This is an automated email from the ASF dual-hosted git repository.

tabish pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/qpid-protonj2.git


The following commit(s) were added to refs/heads/main by this push:
     new 134b2ef5 PROTON-2783 Add missing null check in driver to avoid 
spurious logs
134b2ef5 is described below

commit 134b2ef5dbb3b498f43087c4c58539253855403c
Author: Timothy Bish <[email protected]>
AuthorDate: Thu Dec 14 16:17:33 2023 -0500

    PROTON-2783 Add missing null check in driver to avoid spurious logs
    
    Check for null and avoid verbose logging on connection drops.
---
 .../main/java/org/apache/qpid/protonj2/test/driver/AMQPTestDriver.java  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/protonj2-test-driver/src/main/java/org/apache/qpid/protonj2/test/driver/AMQPTestDriver.java
 
b/protonj2-test-driver/src/main/java/org/apache/qpid/protonj2/test/driver/AMQPTestDriver.java
index 61645bff..39b32cd4 100644
--- 
a/protonj2-test-driver/src/main/java/org/apache/qpid/protonj2/test/driver/AMQPTestDriver.java
+++ 
b/protonj2-test-driver/src/main/java/org/apache/qpid/protonj2/test/driver/AMQPTestDriver.java
@@ -276,7 +276,7 @@ public class AMQPTestDriver implements Consumer<ByteBuffer> 
{
                 if (scriptEntry instanceof ConnectionDropExpectation) {
                     processScript(script.poll());
                     return;
-                } else if (scriptEntry.isOptional()) {
+                } else if (scriptEntry != null && scriptEntry.isOptional()) {
                     script.poll();
                 } else {
                     return;


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to