|
From: Yerubandi R. <Yer...@su...> - 2004-04-01 01:53:03
|
All,
with the build from the latest cvs source code , i found out that =
StatusRequest is not working as intended. i think i have figured out the =
problem .=20
i am including a patch for the fix .=20
Index: MessageServer.java
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
retrieving revision 1.149
diff -u -r1.149 MessageServer.java
--- MessageServer.java 10 Feb 2004 03:12:52 -0000 1.149
+++ MessageServer.java 1 Apr 2004 01:19:18 -0000
@@ -1969,7 +1969,13 @@
status =3D Constants.STATUS_PROCESSED;
} else {
status =3D Constants.STATUS_UN_AUTHORIZED;
- }
+ } =20
+ // local variables ( status, timestamp, cpaId, =
conversationId, service, action)=20
+ // are initialized in this block and a return is being =
made outside the block,=20
+ // .. so if the below return statement is not present , =
always String[] of nulls will be=20
+ // returned and the correct StatusResponse will not be =
generated
+ return new String[]{status, timestamp, cpaId, =
conversationId,
+ service, action};
}
} catch (MessageServerException e) {
exception =3D e;
@@ -1993,8 +1999,8 @@
if (exception !=3D null) {
throw new =
MessageServerException(exception.getMessage());
}
-
- logger.debug("<=3D MessageServer.getMessageStatus");
+ =20
+ logger.debug("<=3D MessageServer.getMessageStatus"); =
=20
}
return new String[]{status, timestamp, cpaId, conversationId,
service, action};
<<MessageServerPatch>>=20
-Venkat
|