Skip to content

0.11.x backport: fix(encryption) [18/N] wire in ags1 file length for tamper proofing #3236 - #3270

Merged
CTTY merged 1 commit into
apache:0.11.xfrom
xanderbailey:xb/backport-3236-0.11.x
Oct 9, 2026
Merged

CTTY merged 1 commit into
apache:0.11.xfrom
xanderbailey:xb/backport-3236-0.11.x

Conversation

@xanderbailey

Copy link
Copy Markdown
Contributor

(cherry picked from commit c9a25b2)

Which issue does this PR close?

  • Closes #.

What changes are included in this PR?

Are these changes tested?

AI Disclosure

@xanderbailey

Copy link
Copy Markdown
Contributor Author

We need #3271 to merge first

@xanderbailey xanderbailey changed the title fix(encryption) [18/N] wire in ags1 file length for tamper proofing #3236 0.11.0 backport: fix(encryption) [18/N] wire in ags1 file length for tamper proofing #3236 Sep 24, 2026
@xanderbailey xanderbailey changed the title 0.11.0 backport: fix(encryption) [18/N] wire in ags1 file length for tamper proofing #3236 0.11.x backport: fix(encryption) [18/N] wire in ags1 file length for tamper proofing #3236 Sep 24, 2026
…pache#3236)

* feat(encryption) [18N] wire in ags1 file length for tamper proofing

* remove debug

* expect

* fix test

* fix opendal

* make more clear

* Add oversized tests

* don't do unreachable

* import fix

* Assert length in streaming tests

* DRAFT try key_metadata_with_length

* fix(encryption): address key metadata length review

(cherry picked from commit c9a25b2)
@xanderbailey
xanderbailey force-pushed the xb/backport-3236-0.11.x branch from d9ac6f9 to afe9610 Compare October 1, 2026 09:35

@dannycjones dannycjones left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. @kevinjqliu / @laskoviymishka, would you be able to lend a hand in reviewing and merging this backport?

@CTTY CTTY left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @xanderbailey for working on this!

@CTTY
CTTY merged commit f98cdf3 into apache:0.11.x Oct 9, 2026
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants