Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Docs | Add ChunkedContentDecoder Documentation with State Machine Diagram #8778

Merged
merged 1 commit into from
Feb 10, 2025

Conversation

shirady
Copy link
Contributor

@shirady shirady commented Feb 9, 2025

Explain the changes

  1. Add ChunkedContentDecoder documentation with state machine diagram.

Issues:

  1. It was a gap mentioned in PR FIx Request Handling on Requests with Trailing Headers (Mainly in ChunkedContentDecoder) #8753, and also continues PR Adding support for chunked upload #5397

Testing Instructions:

  1. none
  • Doc added/updated
  • Tests added

@shirady shirady requested review from guymguym and jackyalbo February 9, 2025 12:54
@shirady shirady self-assigned this Feb 9, 2025
@shirady shirady force-pushed the chunked-content-decoder-docs branch 2 times, most recently from c42f4e0 to 9d07651 Compare February 9, 2025 13:03
@shirady shirady requested a review from jackyalbo February 9, 2025 13:48
@shirady shirady force-pushed the chunked-content-decoder-docs branch from b1e6730 to ea9d1e8 Compare February 9, 2025 14:21
@shirady shirady merged commit b89f7bc into noobaa:master Feb 10, 2025
11 checks passed
@shirady shirady deleted the chunked-content-decoder-docs branch February 10, 2025 06:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants