starred/windmill
1
0
Fork 0
mirror of https://github.com/windmill-labs/windmill.git synced 2026-08-21 00:02:23 +00:00
Code Issues Packages Projects Releases Wiki Activity

cloud debounce keys potential 'value too long' error (#8750)

Browse Source
This commit is contained in:
wendrul
2026-04-07 19:57:48 +02:00
committed by GitHub
parent 8b9523e03c
commit eb32206940
1 changed files with 18 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
backend/windmill-queue/src/jobs.rs
+18 -5
View File
@@ -3640,9 +3640,19 @@ pub fn resolve_debounce_key<'b>(
tracing::debug!("Original debounce key: {}", original_debounce_key);
// If debounce_key is not too long (< 255 chars), keep it as is, otherwise hash it
// If debounce_key is not too long (< 255 chars), keep it as is, otherwise hash it.
// On cloud, we prepend "{workspace_id}:" so we must reserve space for that prefix
// to avoid exceeding the VARCHAR(255) column limit.
const MAX_DEBOUNCE_KEY_LENGTH: usize = 255;
let resolved = if original_debounce_key.len() <= MAX_DEBOUNCE_KEY_LENGTH {
#[cfg(feature = "cloud")]
let prefix = format!("{workspace_id}:");
#[cfg(not(feature = "cloud"))]
let prefix = String::new();
let max_key_length = MAX_DEBOUNCE_KEY_LENGTH - prefix.len();
let resolved = if original_debounce_key.len() <= max_key_length {
original_debounce_key
} else {
let hash = calculate_hash(&original_debounce_key);
@@ -3651,11 +3661,14 @@ pub fn resolve_debounce_key<'b>(
original_debounce_key.len(),
hash
);
hash
if hash.len() > max_key_length {
hash[..max_key_length].to_string()
} else {
hash
}
};
#[cfg(feature = "cloud")]
let resolved = format!("{workspace_id}:{resolved}");
let resolved = format!("{prefix}{resolved}");
tracing::debug!("Final debounce key: {}", resolved);
resolved