Skip to content

feat: add secure peer quick links - #85

Merged
roziscoding merged 5 commits into
mainfrom
feat/quick-link
Aug 19, 2026
Merged

feat: add secure peer quick links#85
roziscoding merged 5 commits into
mainfrom
feat/quick-link

Conversation

@roziscoding

Copy link
Copy Markdown
Owner

Summary

  • add a ConfigSecret-backed external Jack access profile
  • generate dedicated, revocable jack-link:v1 credentials through the management API
  • add secure copy/import flows that prefill the existing editable peer form
  • validate URL credentials, reserved/dangerous/duplicate headers, payload size, and API-key cleanup on failure

Verification

  • 511 tests pass
  • backend and UI typechecks pass
  • lint passes
  • Nuxt production build passes
  • independent security/code review approved the remediated diff

@greptile-apps

greptile-apps Bot commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

The PR adds revocable, credential-bearing peer quick links backed by an external Jack access profile, with management API generation and UI import/review flows.

  • Adds raw and resolved external-access configuration with serialized, field-preserving updates.
  • Generates one-time quick links using newly issued peer API keys and revokes keys when link creation fails.
  • Validates imported links, URLs, headers, credentials, and payload size before prefilling the peer form.
  • Adjusts Radarr manual-import discovery to scan the download folder while associating the movie in the command payload.

Confidence Score: 5/5

The PR appears safe to merge because no blocking failure remains in the eligible follow-up scope.

No blocking failure remains.

Reviews (5): Last reviewed commit: "chore: fix linter errors" | Re-trigger Greptile

Radarr's manual import controller short-circuits on movieId and lists the
files already in the movie's library folder, ignoring the folder we asked it
to scan. Before a first import that folder doesn't exist, so the request blew
up with DirectoryNotFoundException. The movieId is already carried on the
ManualImport command payload, which is where it actually maps files to the
movie.
@roziscoding
roziscoding merged commit 024ac99 into main Aug 19, 2026
10 checks passed
@roziscoding
roziscoding deleted the feat/quick-link branch August 19, 2026 19:35
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