[PR #251] [MERGED] fix http response load when filter #176

Closed
opened 2025-12-29 07:19:40 +01:00 by adam · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/mountain-loop/yaak/pull/251
Author: @hxzhao527
Created: 8/14/2025
Status: Merged
Merged: 9/16/2025
Merged by: @gschier

Base: mainHead: master


📝 Commits (2)

  • 6218ba0 fix http response load when filter
  • 5694e48 Refactor response handling: remove useFilterResponse and useResponseBodyBlob, consolidate filtering logic into useResponseBodyText and improve mimetype parsing

📊 Changes

11 files changed (+97 additions, -102 deletions)

View changed files

📝 src-tauri/Cargo.lock (+11 -0)
📝 src-tauri/Cargo.toml (+1 -0)
📝 src-tauri/src/encoding.rs (+23 -12)
📝 src-tauri/src/lib.rs (+27 -17)
📝 src-web/components/responseViewers/HTMLOrTextViewer.tsx (+2 -2)
📝 src-web/components/responseViewers/TextViewer.tsx (+4 -5)
src-web/hooks/useFilterResponse.ts (+0 -31)
src-web/hooks/useResponseBodyBlob.ts (+0 -12)
📝 src-web/hooks/useResponseBodyText.ts (+9 -5)
📝 src-web/lib/responseBody.ts (+16 -14)
📝 src-web/lib/tauri.ts (+4 -4)

📄 Description

fix


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/mountain-loop/yaak/pull/251 **Author:** [@hxzhao527](https://github.com/hxzhao527) **Created:** 8/14/2025 **Status:** ✅ Merged **Merged:** 9/16/2025 **Merged by:** [@gschier](https://github.com/gschier) **Base:** `main` ← **Head:** `master` --- ### 📝 Commits (2) - [`6218ba0`](https://github.com/mountain-loop/yaak/commit/6218ba096120ebce78cebef05740c3d634db4637) fix http response load when filter - [`5694e48`](https://github.com/mountain-loop/yaak/commit/5694e48baa983d10abc17fac4156bf40058ef673) Refactor response handling: remove `useFilterResponse` and `useResponseBodyBlob`, consolidate filtering logic into `useResponseBodyText` and improve mimetype parsing ### 📊 Changes **11 files changed** (+97 additions, -102 deletions) <details> <summary>View changed files</summary> 📝 `src-tauri/Cargo.lock` (+11 -0) 📝 `src-tauri/Cargo.toml` (+1 -0) 📝 `src-tauri/src/encoding.rs` (+23 -12) 📝 `src-tauri/src/lib.rs` (+27 -17) 📝 `src-web/components/responseViewers/HTMLOrTextViewer.tsx` (+2 -2) 📝 `src-web/components/responseViewers/TextViewer.tsx` (+4 -5) ➖ `src-web/hooks/useFilterResponse.ts` (+0 -31) ➖ `src-web/hooks/useResponseBodyBlob.ts` (+0 -12) 📝 `src-web/hooks/useResponseBodyText.ts` (+9 -5) 📝 `src-web/lib/responseBody.ts` (+16 -14) 📝 `src-web/lib/tauri.ts` (+4 -4) </details> ### 📄 Description fix - https://feedback.yaak.app/p/json-response-filtering-fails-with-non-ascii-characters - https://feedback.yaak.app/p/jsonpath-plugin-returns-values-with-accent-incorrectly --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
adam added the pull-request label 2025-12-29 07:19:40 +01:00
adam closed this issue 2025-12-29 07:19:40 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/yaak#176