Skip to content

Commit c6704b8

Browse files
serrrfiratclaude
andcommitted
fix: skip empty verification token write, consistent with app_id/app_secret
Address zmanian review nit #4: only write verification_token to workspace when present, matching the if-let pattern used for app_id and app_secret. Functionally identical (the auth check filters empty strings), but consistent. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 810cb18 commit c6704b8

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

channels-src/feishu/src/lib.rs

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -310,8 +310,9 @@ impl Guest for FeishuChannel {
310310
if let Some(ref app_secret) = config.app_secret {
311311
let _ = channel_host::workspace_write(APP_SECRET_PATH, app_secret);
312312
}
313-
let verification_token = config.verification_token.as_deref().unwrap_or("");
314-
let _ = channel_host::workspace_write(VERIFICATION_TOKEN_PATH, verification_token);
313+
if let Some(ref verification_token) = config.verification_token {
314+
let _ = channel_host::workspace_write(VERIFICATION_TOKEN_PATH, verification_token);
315+
}
315316

316317
if let Some(owner_id) = &config.owner_id {
317318
let _ = channel_host::workspace_write(OWNER_ID_PATH, owner_id);

0 commit comments

Comments
 (0)