github-starred/yaak
2
0
Fork 0
mirror of https://github.com/mountain-loop/yaak.git synced 2026-05-06 18:16:03 -05:00
Code Issues 71 Packages Projects Releases 101 Wiki Activity

[PR #418] [MERGED] Add redirect drop metadata and warning UI #968

New Issue
Closed
opened 2026-04-20 19:54:21 -05:00 by GiteaMirror · 0 comments
GiteaMirror commented 2026-04-20 19:54:21 -05:00
Owner
Copy Link

📋 Pull Request Information

Original PR: https://github.com/mountain-loop/yaak/pull/418
Author: @gschier
Created: 3/4/2026
Status: ✅ Merged
Merged: 3/5/2026
Merged by: @gschier

Base: main ← Head: feat/redirect-drop-metadata-warning

📝 Commits (3)

  • a42ffd0 Add redirect drop metadata and response warning UI
  • ae98a71 Fix tooltip spacing
  • 7063c35 Preserve request body across 307/308 redirects

📊 Changes

12 files changed (+292 additions, -71 deletions)

View changed files

📝 crates-cli/yaak-cli/src/context.rs (+1 -2)
📝 crates-cli/yaak-cli/src/main.rs (+3 -6)
📝 crates/yaak-http/src/sender.rs (+29 -3)
📝 crates/yaak-http/src/transaction.rs (+57 -18)
📝 crates/yaak-models/bindings/gen_models.ts (+1 -1)
📝 crates/yaak-models/src/models.rs (+4 -0)
📝 crates/yaak-plugins/bindings/gen_models.ts (+1 -1)
📝 packages/plugin-runtime-types/src/bindings/gen_models.ts (+1 -1)
📝 plugins/auth-oauth2/src/index.ts (+0 -1)
📝 src-web/components/HttpResponsePane.tsx (+123 -16)
📝 src-web/components/HttpResponseTimeline.tsx (+28 -3)
📝 src-web/components/core/Tooltip.tsx (+44 -19)

📄 Description

Add a badge to the response pane to notify the user if a redirect caused any request body or headers to be dropped.

CleanShot 2026-03-04 at 15 01 10
  • add explicit redirect metadata for dropped body and dropped headers to HTTP response timeline events
  • emit dropped-body and dropped-header details from redirect handling in crates/yaak-http
  • update response pane warning badge to read redirect drop metadata directly and only show once response is closed
  • improve timeline redirect detail and summary text to show dropped body and headers
  • update shared tooltip positioning to support above/below placement and pointer orientation

Validation

  • npm run -w src-web lint
  • cargo check -p yaak-http -p yaak-models

🔄 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/418 **Author:** [@gschier](https://github.com/gschier) **Created:** 3/4/2026 **Status:** ✅ Merged **Merged:** 3/5/2026 **Merged by:** [@gschier](https://github.com/gschier) **Base:** `main` ← **Head:** `feat/redirect-drop-metadata-warning` --- ### 📝 Commits (3) - [`a42ffd0`](https://github.com/mountain-loop/yaak/commit/a42ffd0e8bbabb40cbf3eb0d0857c75cc00d0391) Add redirect drop metadata and response warning UI - [`ae98a71`](https://github.com/mountain-loop/yaak/commit/ae98a71294ddd650ece4dadb8c5459bc3341456b) Fix tooltip spacing - [`7063c35`](https://github.com/mountain-loop/yaak/commit/7063c356cc5b24ce50d5259ee0adcb1698cdbe49) Preserve request body across 307/308 redirects ### 📊 Changes **12 files changed** (+292 additions, -71 deletions) <details> <summary>View changed files</summary> 📝 `crates-cli/yaak-cli/src/context.rs` (+1 -2) 📝 `crates-cli/yaak-cli/src/main.rs` (+3 -6) 📝 `crates/yaak-http/src/sender.rs` (+29 -3) 📝 `crates/yaak-http/src/transaction.rs` (+57 -18) 📝 `crates/yaak-models/bindings/gen_models.ts` (+1 -1) 📝 `crates/yaak-models/src/models.rs` (+4 -0) 📝 `crates/yaak-plugins/bindings/gen_models.ts` (+1 -1) 📝 `packages/plugin-runtime-types/src/bindings/gen_models.ts` (+1 -1) 📝 `plugins/auth-oauth2/src/index.ts` (+0 -1) 📝 `src-web/components/HttpResponsePane.tsx` (+123 -16) 📝 `src-web/components/HttpResponseTimeline.tsx` (+28 -3) 📝 `src-web/components/core/Tooltip.tsx` (+44 -19) </details> ### 📄 Description Add a badge to the response pane to notify the user if a redirect caused any request body or headers to be dropped. <img width="1040" height="628" alt="CleanShot 2026-03-04 at 15 01 10" src="https://github.com/user-attachments/assets/25876e4c-cfe7-49f6-948f-7efa64c1c8a9" /> - add explicit redirect metadata for dropped body and dropped headers to HTTP response timeline events - emit dropped-body and dropped-header details from redirect handling in crates/yaak-http - update response pane warning badge to read redirect drop metadata directly and only show once response is closed - improve timeline redirect detail and summary text to show dropped body and headers - update shared tooltip positioning to support above/below placement and pointer orientation ## Validation - npm run -w src-web lint - cargo check -p yaak-http -p yaak-models --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-04-20 19:54:21 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
pull-request
Mirrored from GitHub Pull Request
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
GiteaMirror ninjasurge
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/yaak#968
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?