[PR #209] [MERGED] Request Inheritance #134

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

📋 Pull Request Information

Original PR: https://github.com/mountain-loop/yaak/pull/209
Author: @gschier
Created: 5/16/2025
Status: Merged
Merged: 5/23/2025
Merged by: @gschier

Base: masterHead: request-inheritance


📝 Commits (8)

  • 30bb9e9 Define models and migration
  • 05b8b26 Merge branch 'master' into request-inheritance
  • 6dc8270 Editing and basic sending of folder auth
  • 4d47662 Refactor model fields and apply auth everywhere
  • cf9e860 Resolve and send inherited headers/metadata
  • e4dff3f Show inherited auth everywhere
  • cc4e516 Override headers viewer
  • 4546345 Merge branch 'master' into request-inheritance

📊 Changes

41 files changed (+1033 additions, -405 deletions)

View changed files

📝 packages/plugin-runtime-types/src/bindings/gen_models.ts (+5 -5)
src-tauri/migrations/20250516182745_default-attrs.sql (+15 -0)
📝 src-tauri/src/grpc.rs (+8 -3)
📝 src-tauri/src/http_request.rs (+52 -40)
📝 src-tauri/src/lib.rs (+2 -2)
📝 src-tauri/src/render.rs (+2 -2)
📝 src-tauri/vendored/plugins/auth-oauth2/build/index.js (+26 -5)
📝 src-tauri/vendored/plugins/importer-curl/build/index.js (+17 -14)
📝 src-tauri/yaak-git/bindings/gen_models.ts (+3 -5)
📝 src-tauri/yaak-models/bindings/gen_models.ts (+7 -5)
📝 src-tauri/yaak-models/src/models.rs (+70 -36)
📝 src-tauri/yaak-models/src/queries/folders.rs (+40 -2)
📝 src-tauri/yaak-models/src/queries/grpc_requests.rs (+42 -1)
📝 src-tauri/yaak-models/src/queries/http_requests.rs (+42 -1)
📝 src-tauri/yaak-models/src/queries/websocket_requests.rs (+46 -1)
📝 src-tauri/yaak-models/src/queries/workspaces.rs (+15 -2)
📝 src-tauri/yaak-plugins/bindings/gen_models.ts (+3 -5)
📝 src-tauri/yaak-plugins/src/manager.rs (+2 -2)
📝 src-tauri/yaak-sync/bindings/gen_models.ts (+3 -5)
📝 src-tauri/yaak-ws/src/commands.rs (+21 -2)

...and 21 more files

📄 Description

This PR is meant to provide a way for requests to inherit headers or authentication from the workspace or parent folders.

  • Define new Folder/Workspace model fields
  • UI to edit Folder auth
  • UI to edit Folder headers
  • UI to edit Workspace auth
  • UI to edit Workspace headers
  • Logic for inherited auth in HTTP
  • Logic for inherited auth in gRPC
  • Logic for inherited auth in WebSockets
  • Logic for inherited headers in HTTP
  • Logic for inherited headers in gRPC
  • Logic for inherited headers in WebSockets

🔄 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/209 **Author:** [@gschier](https://github.com/gschier) **Created:** 5/16/2025 **Status:** ✅ Merged **Merged:** 5/23/2025 **Merged by:** [@gschier](https://github.com/gschier) **Base:** `master` ← **Head:** `request-inheritance` --- ### 📝 Commits (8) - [`30bb9e9`](https://github.com/mountain-loop/yaak/commit/30bb9e95f5a0b65b03aa0f7cb30cc7a6029c4872) Define models and migration - [`05b8b26`](https://github.com/mountain-loop/yaak/commit/05b8b265b8c96a973fb0de7af8a3c6e6ed5d7968) Merge branch 'master' into request-inheritance - [`6dc8270`](https://github.com/mountain-loop/yaak/commit/6dc827055dd09bfe5db7e8ab9114092a8cca3d96) Editing and basic sending of folder auth - [`4d47662`](https://github.com/mountain-loop/yaak/commit/4d47662717bcd9b66bec7f86d5d77c28b1728344) Refactor model fields and apply auth everywhere - [`cf9e860`](https://github.com/mountain-loop/yaak/commit/cf9e86071d907d3c4fe52343a3b109620741a8a7) Resolve and send inherited headers/metadata - [`e4dff3f`](https://github.com/mountain-loop/yaak/commit/e4dff3f1d713f35dfe0c38c397008b0ae8721600) Show inherited auth everywhere - [`cc4e516`](https://github.com/mountain-loop/yaak/commit/cc4e516e2c458dd145b0b6a19599a508e9bde95b) Override headers viewer - [`4546345`](https://github.com/mountain-loop/yaak/commit/4546345efea0ac1e30fe970290254e1021a9fcfc) Merge branch 'master' into request-inheritance ### 📊 Changes **41 files changed** (+1033 additions, -405 deletions) <details> <summary>View changed files</summary> 📝 `packages/plugin-runtime-types/src/bindings/gen_models.ts` (+5 -5) ➕ `src-tauri/migrations/20250516182745_default-attrs.sql` (+15 -0) 📝 `src-tauri/src/grpc.rs` (+8 -3) 📝 `src-tauri/src/http_request.rs` (+52 -40) 📝 `src-tauri/src/lib.rs` (+2 -2) 📝 `src-tauri/src/render.rs` (+2 -2) 📝 `src-tauri/vendored/plugins/auth-oauth2/build/index.js` (+26 -5) 📝 `src-tauri/vendored/plugins/importer-curl/build/index.js` (+17 -14) 📝 `src-tauri/yaak-git/bindings/gen_models.ts` (+3 -5) 📝 `src-tauri/yaak-models/bindings/gen_models.ts` (+7 -5) 📝 `src-tauri/yaak-models/src/models.rs` (+70 -36) 📝 `src-tauri/yaak-models/src/queries/folders.rs` (+40 -2) 📝 `src-tauri/yaak-models/src/queries/grpc_requests.rs` (+42 -1) 📝 `src-tauri/yaak-models/src/queries/http_requests.rs` (+42 -1) 📝 `src-tauri/yaak-models/src/queries/websocket_requests.rs` (+46 -1) 📝 `src-tauri/yaak-models/src/queries/workspaces.rs` (+15 -2) 📝 `src-tauri/yaak-plugins/bindings/gen_models.ts` (+3 -5) 📝 `src-tauri/yaak-plugins/src/manager.rs` (+2 -2) 📝 `src-tauri/yaak-sync/bindings/gen_models.ts` (+3 -5) 📝 `src-tauri/yaak-ws/src/commands.rs` (+21 -2) _...and 21 more files_ </details> ### 📄 Description This PR is meant to provide a way for requests to inherit headers or authentication from the workspace or parent folders. - [x] Define new `Folder`/`Workspace` model fields - [x] UI to edit Folder auth - [x] UI to edit Folder headers - [x] UI to edit Workspace auth - [x] UI to edit Workspace headers - [x] Logic for inherited auth in HTTP - [x] Logic for inherited auth in gRPC - [x] Logic for inherited auth in WebSockets - [x] Logic for inherited headers in HTTP - [x] Logic for inherited headers in gRPC - [x] Logic for inherited headers in WebSockets --- <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:29 +01:00
adam closed this issue 2025-12-29 07:19:29 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/yaak#134