<?xml version="1.0" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://www.jacorb.org/bugzilla/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.2"
          urlbase="https://www.jacorb.org/bugzilla/"
          
          maintainer="jacorb@goots.org"
>

    <bug>
          <bug_id>1012</bug_id>
          
          <creation_ts>2015-07-08 13:56:25 +0000</creation_ts>
          <short_desc>Missed send_request while completing a forward request</short_desc>
          <delta_ts>2015-10-22 04:41:22 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>JacORB</product>
          <component>ORB</component>
          <version>3.6.1</version>
          <rep_platform>PC</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P5</priority>
          <bug_severity>major</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Hugo Roenick">hroenick</reporter>
          <assigned_to name="Mailinglist to track bugs">jacorb-bugs</assigned_to>
          <cc>hroenick</cc>
    
    <cc>jacorb</cc>
    
    <cc>mesnierp</cc>
    
    <cc>vgpinheiro</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>3321</commentid>
    <comment_count>0</comment_count>
      <attachid>466</attachid>
    <who name="Hugo Roenick">hroenick</who>
    <bug_when>2015-07-08 13:56:25 +0000</bug_when>
    <thetext>Created attachment 466
demo that demonstrate the suspicious behaviour

I stumbled in another unexpected behaviour while isolating the problem reported in Bug 1011 . Instead of making 2 or more calls inside the send_request interception point, just make 1 more call that is also going to receive an exception and resolve it through a forward_request. In this case I noticed that there is an extra send_request call and the previous one apparently is not completed (does not arrives in server side). 

Let me try to make myself clear with an example:

[client]											[server]
	---- calls A() ----&gt;
	&lt;---- NO_PERMISSION (op = A) ----
	
	receive_exception:
		do forward_request( op = A )

	---- calls A() ----&gt;
		---- send_request( op = A )  ----&gt;
			---- calls B() ----&gt;
			&lt;---- NO_PERMISSION (op = B) ----
			
			receive_exception:
				do forward_request( op = B )
			
			---- calls B() ----&gt;
			&lt;---- successful reply (op = B) ----
		--- complete the send_request( op = A) ---&gt;
	// &quot;apparently this completed send_request does not arrives to the server&quot;
	// why ???

	// an extra call is sent. Why??
	---- calls A() ----&gt;
	&lt;---- successful reply (op = A) ----</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>3322</commentid>
    <comment_count>1</comment_count>
    <who name="Hugo Roenick">hroenick</who>
    <bug_when>2015-07-08 13:56:46 +0000</bug_when>
    <thetext>Through the provided demo this can we noticed by the following prints:

send_request: canIComeIn
canIComeIn -&gt; secret = Any wrong secret
receive_exception: canIComeIn

send_request: canIComeIn
	send_request: itsMe
	itsMe -&gt; secret = Any wrong secret
	receive_exception: itsMe
	
	send_request: itsMe
	itsMe -&gt; secret = This is truly the real one!
canIComeIn -&gt; secret = This is truly the real one!

// this print below is the extra call
send_request: canIComeIn
canIComeIn -&gt; secret = This is truly the real one!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>3328</commentid>
    <comment_count>2</comment_count>
    <who name="Nick Cross">jacorb</who>
    <bug_when>2015-07-31 09:24:28 +0000</bug_when>
    <thetext>
Hugo - would it be possible to verify current git head to ensure that it is fixed there?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>3330</commentid>
    <comment_count>3</comment_count>
    <who name="Hugo Roenick">hroenick</who>
    <bug_when>2015-08-03 05:32:48 +0000</bug_when>
    <thetext>Will test it and get back to you as soon as possible.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>3335</commentid>
    <comment_count>4</comment_count>
    <who name="Vinicius Pinheiro">vgpinheiro</who>
    <bug_when>2015-08-19 14:48:05 +0000</bug_when>
    <thetext>I&apos;ve already informed this on the jacorb-dev list, but just for the record i&apos;ll duplicate the info: this bug was not fixed by the current git head.

I submitted a pull request to reproduce it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>3351</commentid>
    <comment_count>5</comment_count>
    <who name="Phil Mesnier">mesnierp</who>
    <bug_when>2015-10-01 17:18:30 +0000</bug_when>
    <thetext>Here&apos;s the deal with the extra call. In the regression test ExtraCall case, the itsMe operation gets a NO_PERMISSION which triggers the forward. This causes the Delegate to rebind, closing the existing connection and making a new one. The itsMe call completes allowing the original canIComeIn request to proceed. At this point, the Delegate throws a RemarshalException from within _invoke_internal because the connection it expected to use no longer exists. 

So I will adjust the regression test to use all three scenarios but I think this issue is resolved.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>3362</commentid>
    <comment_count>6</comment_count>
    <who name="Phil Mesnier">mesnierp</who>
    <bug_when>2015-10-22 04:41:22 +0000</bug_when>
    <thetext>with commit b1c67af0d4efb973e76aa2dcbce593b4eecef3d9 the regression test fits the behavior of the Delegate which is correct given the connection manager&apos;s closure of unbound connections. 

It may be possible to continue caching otherwise unbound connections for some time but that will involve reworking the giop connection manager.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="0"
              isprivate="0"
          >
            <attachid>466</attachid>
            <date>2015-07-08 13:56:25 +0000</date>
            <delta_ts>2015-07-08 13:56:25 +0000</delta_ts>
            <desc>demo that demonstrate the suspicious behaviour</desc>
            <filename>door-demo.tar.gz</filename>
            <type>application/x-gzip</type>
            <size>4259</size>
            <attacher name="Hugo Roenick">hroenick</attacher>
            
              <data encoding="base64">H4sIACFknVUAA+082W7suLFGcIPg+gPu00XA6TsB5Blbvdtzvd342J6kB8cLbE9mkAUNWWK35aMW
FUrykoG/Jr+TlzzkB5KH/Mat4qKtpV48ts9kIsK9iCwWi6yFVUW6Hca4QyesufJ6pdVqbfX7RHxu
ys9Wpyc/VSHtTr+31dls9Xst0mp3N3utFdJ/RZqSEoeRxYGUG86o79ofquAAbDSagUfNI/n8FymO
5r8Z0ihy/XH48pKwDP97nT7wv9/frPn/JiXhv+t4r2UDltL/Lup/r9faqvn/FiXV/2HAJubDxHv5
MWA9Nnu9av53QecL+t/d6q2Q1suTMl3+zfm/8tP/+tnKT1ZWTiybnF2Sb4kqWLfyn/DqwOvP8MLn
vy2G8uDq6kJ9xR5/UZiy5Seq/qcrKz+3Qe4mls3ZxIpoCPuQR+3IZf6F5Y+pgv8rvP5jZeW/C7B3
buhee3TgO/RhZWWrtd3rttudN5GbH0lJ9P/VtH+u/vfam1P7fxv3/1r/X7/sBpzdgr4R4L0f7jVu
oijYbjYn1h31TSuw7BtqMj5unp+dNHtmy2w1JOT2Q+gm0Pf39+Z9V8B1Wq1289uT95fQcWJtuD6s
m2/Txioh0GM7FNXvmW2hhi8wHKmCEBUbd71ha9gyH0KnsQ9D7E6YQ73fUB4C9n2BYLeZq1tFqDFn
cTBw9lHwd5v6CVssHrkjy47gceBHlNs0iBh/F493m5kmhLxTCNs4hH7ABiDxgzUGT3r/1uK7zfRR
DO1bE7p/BOOSDVIcQbQJKGBKQGE8GiJKqLi17iwzHXJzt5mr0UA249dJZdfcNNsImKuVoODt3Wbw
dRAwVyfBlHCY17HrOWbIYiD32LeZg/MZXJ5tfPFF/383oO9MQJxRMzslrBCQahgJf+RyQMH4437I
7d1msXJVkeTFsJqhekwq9lf1zpVwF6REC4yUHtU1x3HVJ8NcKViw0QSuR/mG7FRkv+qmV6tjds1O
TgwUgM38kTuOuRD3tD6Z8/6n32UZ+aSnnQMFAzWm0TSoqs+CarIP+DieUD/a3/j2mrHI9qwwDKzo
phlsf/qdjvRMD/TQu6ABC11c4qcmKpaUF/2Bg2bl50k1bEw1mELcpwhI16JZuhggGYqBCzEURIre
WHFoTtgtm8NJFOgNyc6FuPjpd1kVeCrhZ9rHoQEFr8O3Ex0tNjxmqwuzkEtXQn5xCmqxga5EIssn
UZhIgTclU8GVL6M1U6t1da4sa9L29bySimmJTxUaJlWi5ZXikrbQB2rHWJlf+aS6sCrBjRXS/TH1
KaCiG3JEsAKyvsAkZnlhvk7VJgiAa/iY79ec6rjbLKEnUxlWKIB+DKXZVGZytVTkSoRtETGbFrAy
mVpYmnJyRFanRSjZHmAeH9vfqUu+JP4/7Lk/hPyP9P97EMfV+Z+3KDn+v9JBwFL877VF/qffrfn/
FmWa/1jzskKwBP872NBqb3bb/Zr/b1Gm+X9JOWzvF/SPMQ2jTIg6mASeiRHI0mPMyf90sTFn/zu9
Nub/6/zP6xeZnqAEWb+zuupOAsYjkjqPJr7OLt7tZJvYZGweQuWBeeA/VrQcQgjgUfR0LyMrisMK
sMHp1fHF6cH7imbMFHln1+g8VkCcng3Pjy9OBpeXg7PTKZjB2TlQ4lC7tAVF3bXpIQMpf5ge4Bwe
LJFeTpTA/JLxe4s7Sj0W6lJQqBF7Tq+kAZg0cn3LI0F87bk2EaE9maW0BOYG3nhIMotJXOQOBugh
OOxVvcl3GHgE3L2DuIdcRtz1xwSzVDuZauACTOQ6WyVWnNhy3bFekjqLSENhD8XHusa6nsW1BvRg
wBHduKGJZJA9Bb+T1kMnqFYEqTrRG2oVRYQ8IVW/PANquOvQlMI75joEQlHq3tEhl4QaU/wj3F0D
xJzdhyQvD4rA68eI/u4PhGICjDqHnEIoFLnAsz3ix54nKcsLH1GjFWr3VDDGXXNMI03SMJRQQ1uC
GekQquPAMe8sL6ZrcqwySkoHNBXGoWNFluwLKk4sVHOxnPxRzZFgpV5TE17wORRjGlOjAReT77+m
XkC5GT0G1FhT5D2Jd9uK7BtiHD+gVIDdIHQtGUtJx4SGIVqrPdL42qcPAUgydQhwEWRVkIBAdjJW
Y0d1F8wiPr0n2t4YClWOgsz62NmlmqIeVohDsGzAEigE7ogYn4BFujg+vBpeHh9eHF9JDpiwzJYX
GilCM6TwEK2ls0vJy1kzo4XinzejYPVOzt8fXx0fDU/PMsQvI9FF8QnLRXw1Ia1EzOcNGYLFGVLN
yiV1aCHkLLqh/DUQcxp4jxWI56NwAJyzR2MObMaYGloQQChi7qfmTVqqp1fNmVT4/0fwdui5ILDP
c/lyZY7/1+lt9vP+f6fdwfi/9v9ev2T9PxNZnzqB4rTBZea7eDSiYL0uqOVQvlNs/tL1aGllGXwc
uZ55nhwF5T3O1KkTDqduEqQdCdKKNdIcA6jSKukZSZcoFWHlxkgQ4GMEH9gKCjfamWq5AgU+tEJK
Ihved6a7CjWfWK5vyOGkKsN+b/FxmNiedBuTyp1OmuBRWIi+ABj8tNpQ1ly04m081fRoNKYW5xBn
2FgnjWlHvbEEmkugGzYX5it8ajsqwZpALoC+zJsFFAPfxd3P/RPlYjizoYb7nDRUM8wI+ZKBNAUv
0fc5FZZSja68Q1hD+Ga6AG/g4q9LstZ2ZCodhZCM1ELjg9FACQf55Hoaedkm11xB5+sNjUA9jjQd
yooDQuh3zU0O7e9dn2peXuMEWCieRUWBBdIXZ9e30mU1pTc7jNiQiSYDMCtUzSaJQxws9u0ban+g
sHdwdBtuwYKRiBFLyKWbLnpILHGPJhT9tcDDrHwbPahUf0yN0hlKnAaMroaFHsJXVP0SWvThDKjJ
DVqP0IXVIhFus6g2CkyoEPTXOmV+w/iHr4DgL2GRdqqAjnDTPWeBcjmLzcfofB1a6EdrEiVxpm35
A3CYQPIkA56mlZf68SRVcamaWaKkDiRDyMcsQa+9Ib9xmbv/v0AiaM7+v9XZKuT/O/3NXr/e/9+i
zNv/xYYNym7HnKM3aEVs4trmgfh4x5hHLb9qF69ODr1s1kd7AdWuxEtmhZLO53LpzIEPgZ7rXEFE
CwGHvvWyUJKnoGILpoaqFPNZCaycT5UPYPP+VhoEV9UnuYcqgMRdK8lgzbM4ZVms1TSLRar6vlgO
S9XDBgu1uMfAntTaKbS4UXhCiw05ZUEwzpxY7MHoV+RajZHlhXQt63XOW5iPkDkrrCZ4Z5cipaFJ
YYEeUKwJ9G+nWRIW6JRII92vG5lsiCvGw9rPP9cJHPA4kB4Cfxa5h/3a4iz2HfR70P9oJBt2Q7gf
pkYF47nkF6RL9oAn6RBJrN3A7NY9Z7h+YgpJyuhJfVIv8RMy/coyPcWeQPPx6VGG2kymSWAtrIaQ
neJCiEq9Dk+rySrirDqFWc2ZU2bkQo/q2cxNLamMicyUTpmzuZnSRGCovOaDvqZrJo/aj758DCM6
MVkcmSB7fuT5RiM78jZpQBiRdMvk5BJtM/FOksXpge9cgqRGPKbrRGpbuoLhvStykJn0h3A/s5Ij
3NGsy7idNJGMskIsMDI/+Mz+MBTvRuOc+v6jDjxkuYaA4cNOHrV2Nr8HWpC8DigymTDwzwWYeret
OIQAFFXmOh4/f4AXmGaisdt5woWaCzoJGMuGyLQ2cmJGkjtUZr4rmBOfRRAgQYyG144dcn9jReTG
CgLq08RYBJzeuSwOCfOpWT0poXlG4/KGek4a95ZM6Cmrmypbnsu0Y2QHeggWcwhNRj52lCoKUKkV
LcpxifSnCgPRM9nYT9GgIqgE887MpDbuPWmbUdFHRYiuH0KQb+Rmtp7Bp+PdioOP6cODshMJeZQg
W9RRQm7AstOCcudryaMD6n/vo4OlTnMQW76p+gRnfXqpdGjumpbjVJ4JlZKgTd7Ch2BpCv8Zxr3E
ak/hlaY7b/Qz9jvfMM9zSIx9qEx8jkcKnyIhczwxdJ0Ud0loYWJ76XlNft6IXV7Kzh1tLbTQ8jhj
uUVeFLc80SjHveChSMA879kYXvxMZPXpY8fNP5YyI//z/As/hTIn/9Paam8V8j9QVed/3qTMyP8k
pyznZwfVRyy52FxBSzWvsgZZbzGj8mszdg0BvE7yH7hviI46kVzW85sbhkEjGlf6f435+570+hYk
6/eNQYTYNSXojP2+MXuUa5UGyOaqZwxx9uETvAOA2QUSQMeQftJIUvPSNPI4sYrLZ6Zn6X/mGOZ7
mYE5+t9rbRXv/3U6W/X9vzcpM/S/cAoYVB/roSuAdqAy2zu4Ghy8H/z2+Bn53tm5WtHypSX+d6gU
4Pj08OxocPqr4eHRxRAeDs4vy+Gq0rbFUXTW92sfzNC9v1S6N7Ncy8ArJVx2CE3pURyA4YHo81T4
TznzPutK3gybnxI1/3JhfhJya2h+9hk5sW7x1pZF8J+J/sGIwzAAdEdAqgMTIZ81M5lGdXonCcEo
k5Qwdnhy8NXZxfA3xxfouuusoxjL9V98rMFpbqzOzqxtJeB0KI6pM/wBkz5i2vYXNEm0iS97xCi0
rYlGFainOV4MoEWSQdQZGsHadOyNTSJ0tIU/n0SQmbNjceI9N/OcZEQaVWCN9WQmeIgPMzHW1lVO
OsmoJPSEQhor6Zl5fzSlpQpsAVpyqYWc3iyZUeB07MJmLm8HZI7kS9MK2jqWJBZm+ioBE4tUIVW6
N8g/ovuMHHLXIvHEIvhP+X/3bXgCPZASZEGUxwke9rl/ApUgAfVYmFKOWkJDkSxTyH4ZWNyaiMHE
G59Y/5SKBbQoEOWgqDFk76yiKeHNCG2VdmQNsOSYftiT0iPqhiNZmdy2SNNB+ks2A6NbjRL1VrmX
RKjmWpv1+UZCU5VVRbVE2SnpVKGoMzTlZQJa2IKWFFExjEh64UDLC+aP6hrEv22Z4f9LO/oG9z9b
m5sF/7/d2azvf75JWeD+J158O4ujII7AqlBrMnXZ8xyD1W+4G32f254Vvq0SQpGCmAsy7zqohCy7
Dlrf6ZyN/gdzpxOzSwGzACrht6kvLcK0wEth3p30tWG4YXI3MDQaF4xF0KmhBwQ0ZnRDh1B3Yvmg
BNxYM/HKJDoniQ+R5LhQORSDdZ1mr5SZGXc7cSh0bi3wtiOWUmUg0rK7pIJNAgPCy6sa6lamkrxE
5UhG7pLK5MZqVnGz11/1GoRJqim9agqVxVursLJxcqmy3vl/PCX3+582m0yYb8LXFx1jXv6/2ynk
/9qb3a1uvf+/RZkwJ/bk9i/3NJtBdEhiP3THeGfCw2tEJYfTeMftAZjVx9nu4O2LxruvWy1p/zOI
pOkq3DDCeORKXeeKeOw9insZYKI8vJPxiYpFxBv0j8GMZu4JJHd1svcnVGCyswp/tXVaouT0X24P
L6z98/W/0+kW9b/Vq3//9U3KlP6LnM/Isqn8jxGtbVMnd66vNTBzBkeyh2gVEGUHYLX6fqxS8vvv
6MdS23ODkJq3Dv5LOKfgJdJR+Mwx5u7/7W7x/K/frc//3qT8z1VMyVexR1pbpPXFdvuLbSD+3cUV
6bTa/dVSUdA/7ieSrmPqqytP554VYR54r21urupOQRqA6Z+P22vPQSt/oE+gmTc+xGAu/rysAP7Y
S1mXutSlLnWpS13qUpe61KUudalLXepSl7rUpS4/qPL/diD3cwB4AAA=
</data>

          </attachment>
      

    </bug>

</bugzilla>