pjfanning opened a new pull request, #3521:
URL: https://github.com/apache/pekko/pull/3521
### Motivation
Backport of #3510 to 1.7.x: a compression table advertisement carried an
unbounded number
of entries, each of which is resolved on the inbound control stream — for
actor refs that
means parsing a path and populating the resolve cache — bounded only by the
frame size.
### Modification
Cherry-pick of 6751ab98b7. Conflicts were confined to import blocks in
`ArteryMessageSerializer` (1.7.x also imports `Flush`/`FlushAck` there) and
its spec
(adding `ArteryControlFormats` to an existing import); the change itself
applied clean.
### Result
Same as #3510: an advertisement carrying more entries than the receiver's
configured
`pekko.remote.artery.advanced.compression.<table>.max` is rejected as a
serialization
failure; a legitimately sized one is unaffected, and no bound is applied
when compression
is off locally.
### Tests
- `sbt "++ 2.12.21 remote/Test/compile"` — clean, validating Scala 2.12
- scalafmt clean on the cherry-picked files (the one flagged file is
pre-existing
formatting drift in `NestedPayloadDepthSpec` on 1.7.x, left untouched)
- Test suites intentionally left to CI per the release-prep flow; the
boundary test from
#3510 (exactly `max` accepted, `max + 1` rejected) is included
### References
Backport of #3510.
--
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]
