GitHub user jgallimore opened a pull request:
https://github.com/apache/tomee/pull/272
TOMEE-2234 - WIP
This PR introduces Arquillian tests for BMP finder methods, and addresses
an issue when there are more than 256 handles in the stack. The root issue is
that when proxies are copied, an `inputstream.read()` and
`outputstream.write()` are used to store the handle id, which reads and writes
a single byte respectively. This PR flips that to an int.
Local beans do not get cloned in this way, and therefore are not affected
by this issue.
Please note that the arquillian tests fail in `webapp` mode and that needs
to be resolved before this can be merged in.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jgallimore/tomee TOMEE-2234
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/tomee/pull/272.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #272
----
commit 5fa2b74de94757a3202b79c8b31fefd17e1185d2
Author: Jonathan Gallimore <jon@...>
Date: 2018-12-11T14:49:24Z
TOMEE-2234 add remote test
commit c4391777ca5b074a9984aa57072208745c6c9d45
Author: Jonathan Gallimore <jon@...>
Date: 2018-12-11T21:22:19Z
Merge remote-tracking branch 'apache/master' into TOMEE-2234
commit 9a68c9cc91a211ab64e79378e15426f8d8f3c5cb
Author: Jonathan Gallimore <jon@...>
Date: 2018-12-12T10:41:32Z
TOMEE-2234 serialize the handle as an int as opposed to a byte
----
---
