[PR #487] [MERGED] Add configurable gRPC and WebSocket message size limit #3336

Closed
opened 2026-07-15 02:02:31 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/mountain-loop/yaak/pull/487
Author: @gschier
Created: 6/29/2026
Status: Merged
Merged: 6/30/2026
Merged by: @gschier

Base: mainHead: followup-message-size-setting


📝 Commits (2)

  • 961ad36 Add request message size setting
  • 7a34ba4 Merge remote-tracking branch 'origin/main' into followup-message-size-setting

📊 Changes

34 files changed (+1628 additions, -973 deletions)

View changed files

📝 apps/yaak-client/components/HttpAuthenticationEditor.tsx (+60 -11)
📝 apps/yaak-client/components/ModelSettingsEditor.tsx (+334 -47)
📝 apps/yaak-client/components/Settings/SettingsGeneral.tsx (+26 -61)
📝 apps/yaak-client/components/WebsocketResponsePane.tsx (+15 -3)
📝 apps/yaak-client/components/WorkspaceSettingsDialog.tsx (+3 -1)
📝 apps/yaak-client/components/core/PlainInput.tsx (+7 -4)
📝 apps/yaak-client/hooks/useAuthTab.tsx (+157 -120)
📝 apps/yaak-client/lib/requestSettings.ts (+24 -4)
📝 apps/yaak-client/package.json (+2 -2)
📝 apps/yaak-client/vite.config.ts (+1 -0)
📝 apps/yaak-proxy/package.json (+2 -2)
📝 crates-tauri/yaak-app-client/src/lib.rs (+15 -9)
📝 crates-tauri/yaak-app-client/src/ws_ext.rs (+33 -1)
📝 crates/yaak-git/bindings/gen_models.ts (+4 -0)
📝 crates/yaak-grpc/src/client.rs (+11 -5)
📝 crates/yaak-grpc/src/manager.rs (+82 -22)
📝 crates/yaak-grpc/src/reflection.rs (+7 -3)
📝 crates/yaak-models/bindings/gen_models.ts (+6 -1)
📝 crates/yaak-models/guest-js/store.ts (+1 -1)
crates/yaak-models/migrations/20260629000000_request-message-size.sql (+7 -0)

...and 14 more files

📄 Description

Adds an inherited request message size setting for gRPC and WebSocket requests so large payload handling can be configured from workspace, folder, or request settings.

  • Adds a 64 MiB default Message Size Limit setting, with 0 disabling the limit
  • Applies the resolved setting to gRPC decoding and encoding, including reflection clients
  • Applies the resolved setting to WebSocket receive frame/message limits and outbound sends
  • Adds the model migration and generated TypeScript binding updates

🔄 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/487 **Author:** [@gschier](https://github.com/gschier) **Created:** 6/29/2026 **Status:** ✅ Merged **Merged:** 6/30/2026 **Merged by:** [@gschier](https://github.com/gschier) **Base:** `main` ← **Head:** `followup-message-size-setting` --- ### 📝 Commits (2) - [`961ad36`](https://github.com/mountain-loop/yaak/commit/961ad36c736e08bed79baedb9962c020940d5d9a) Add request message size setting - [`7a34ba4`](https://github.com/mountain-loop/yaak/commit/7a34ba45c2022a82d6029762bb6d91245abb5f32) Merge remote-tracking branch 'origin/main' into followup-message-size-setting ### 📊 Changes **34 files changed** (+1628 additions, -973 deletions) <details> <summary>View changed files</summary> 📝 `apps/yaak-client/components/HttpAuthenticationEditor.tsx` (+60 -11) 📝 `apps/yaak-client/components/ModelSettingsEditor.tsx` (+334 -47) 📝 `apps/yaak-client/components/Settings/SettingsGeneral.tsx` (+26 -61) 📝 `apps/yaak-client/components/WebsocketResponsePane.tsx` (+15 -3) 📝 `apps/yaak-client/components/WorkspaceSettingsDialog.tsx` (+3 -1) 📝 `apps/yaak-client/components/core/PlainInput.tsx` (+7 -4) 📝 `apps/yaak-client/hooks/useAuthTab.tsx` (+157 -120) 📝 `apps/yaak-client/lib/requestSettings.ts` (+24 -4) 📝 `apps/yaak-client/package.json` (+2 -2) 📝 `apps/yaak-client/vite.config.ts` (+1 -0) 📝 `apps/yaak-proxy/package.json` (+2 -2) 📝 `crates-tauri/yaak-app-client/src/lib.rs` (+15 -9) 📝 `crates-tauri/yaak-app-client/src/ws_ext.rs` (+33 -1) 📝 `crates/yaak-git/bindings/gen_models.ts` (+4 -0) 📝 `crates/yaak-grpc/src/client.rs` (+11 -5) 📝 `crates/yaak-grpc/src/manager.rs` (+82 -22) 📝 `crates/yaak-grpc/src/reflection.rs` (+7 -3) 📝 `crates/yaak-models/bindings/gen_models.ts` (+6 -1) 📝 `crates/yaak-models/guest-js/store.ts` (+1 -1) ➕ `crates/yaak-models/migrations/20260629000000_request-message-size.sql` (+7 -0) _...and 14 more files_ </details> ### 📄 Description Adds an inherited request message size setting for gRPC and WebSocket requests so large payload handling can be configured from workspace, folder, or request settings. - Adds a 64 MiB default `Message Size Limit` setting, with `0` disabling the limit - Applies the resolved setting to gRPC decoding and encoding, including reflection clients - Applies the resolved setting to WebSocket receive frame/message limits and outbound sends - Adds the model migration and generated TypeScript binding updates --- <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-07-15 02:02:31 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/yaak#3336