pjfanning opened a new pull request, #3520:
URL: https://github.com/apache/pekko/pull/3520
### Motivation
Backport of #3508 to 1.7.x: gossip lookup-table indexes went from the wire
straight into
`Vector.apply`, so an out-of-range index raised `IndexOutOfBoundsException`
— and for a
`GossipEnvelope` the deferred parse puts that throw inside the cluster
daemon rather than
on a deserialization thread.
### Modification
Cherry-pick of the #3508 change commit. The only conflict was the import
block of
`ClusterMessageSerializerSpec` (the `NotSerializableException` import);
everything else,
including the new `lookup` helper, applied clean — the touched
`.asScala...to(...)` lines
are identical on 1.7.x, where the file already imports
`pekko.util.ccompat.JavaConverters`.
Note #3508 is still open against `main`; if review there changes the fix,
this backport
should be updated to match before merging.
### Result
Same as #3508: gossip referring to a table entry the sender did not include
is reported
as a serialization failure naming the index and the table size.
### Tests
- `sbt "++ 2.12.21 cluster/Test/compile"` — clean, validating Scala 2.12
- `sbt "cluster/scalafmtCheckAll"` — clean
- Test suites intentionally left to CI per the release-prep flow; the three
tamper tests
from #3508 are included, covering `Gossip`, `GossipStatus`, and the
deferred
`GossipEnvelope` path
### References
Backport of #3508.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
