Skip to content

Allow validated Obsidian links in messages - #6681

Closed
tridvm-holistics wants to merge 1 commit into
block:mainfrom
tridvm-holistics:fizz/obsidian-deep-links
Closed

Allow validated Obsidian links in messages#6681
tridvm-holistics wants to merge 1 commit into
block:mainfrom
tridvm-holistics:fizz/obsidian-deep-links

Conversation

@tridvm-holistics

Copy link
Copy Markdown

Summary

  • preserve only strictly validated obsidian://open Markdown links
  • open validated links through Tauri’s native OS opener
  • scope the Tauri permission to Obsidian open URLs
  • add sanitizer, validation, and browser click-path regression tests

Verification

  • pnpm test (5,402 passed)
  • pnpm check
  • pnpm typecheck
  • cargo check --manifest-path desktop/src-tauri/Cargo.toml --package buzz-desktop
  • pnpm exec playwright test tests/e2e/messaging.spec.ts --grep "validated Obsidian link opens"

Co-authored-by: Tri Duong <tri.dvm@holistics.io>
Signed-off-by: Tri Duong <tri.dvm@holistics.io>
@tridvm-holistics
tridvm-holistics requested a review from a team as a code owner August 24, 2026 10:10
@tridvm-holistics
tridvm-holistics deleted the fizz/obsidian-deep-links branch August 24, 2026 10:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant