Skip to content

Handle lossy conversion when proxying response text#2

Merged
vansour merged 1 commit intomainfrom
codex/refactor-process_shell_script_bytes-and-process_html_bytes
Oct 23, 2025
Merged

Handle lossy conversion when proxying response text#2
vansour merged 1 commit intomainfrom
codex/refactor-process_shell_script_bytes-and-process_html_bytes

Conversation

@vansour
Copy link
Copy Markdown
Owner

@vansour vansour commented Oct 23, 2025

Summary

  • avoid cloning shell script response bodies while decoding and rewriting URLs
  • apply the same lossy handling path for HTML responses so proxy rewrites still occur

Testing

  • cargo test

https://chatgpt.com/codex/tasks/task_e_68f984cc41f08323a00f2dd3b6d500cc

Summary by CodeRabbit

  • Refactor
    • Streamlined UTF-8 error handling and recovery mechanisms for improved code maintainability and consistency in text processing operations.

Repository owner deleted a comment from coderabbitai Bot Oct 23, 2025
@vansour vansour merged commit 1411899 into main Oct 23, 2025
1 check was pending
@vansour vansour deleted the codex/refactor-process_shell_script_bytes-and-process_html_bytes branch October 23, 2025 02:36
vansour added a commit that referenced this pull request Dec 7, 2025
…t_bytes-and-process_html_bytes

Handle lossy conversion when proxying response text
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.

1 participant