Revert "fix: video content api Priority use url field"

[seefs001]

Reverts #2691

Summary by CodeRabbit

• Refactor
  • Simplified internal video proxy implementation by removing an external dependency, improving code efficiency while maintaining existing functionality.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Caution

Review failed

The pull request is closed.

Walkthrough

This pull request removes the lo helper library dependency from the video proxy controller by eliminating a ternary conditional. The default case logic for videoURL assignment is simplified to unconditionally use task.FailReason instead of preferring task.Url when available.

Changes

Cohort / File(s)	Summary
Video Proxy URL Selection Logic controller/video_proxy.go	Removed lo import and replaced lo.Ternary(task.Url != "", task.Url, task.FailReason) with direct assignment of task.FailReason to videoURL. Changes default channel type control flow to unconditionally use failure reason instead of preferring valid URL when present.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• fix: video content api Priority use url field #2691: Directly inverse relationship—that PR introduced the lo.Ternary logic to prefer task.Url, while this PR removes it and reverts to unconditional task.FailReason assignment.

Poem

🐰 Away with dependencies we no longer need,
A simpler path where FailReason takes the lead,
No ternary tricks, just straightforward code,
The proxy hops lighter down its road! ✨

✨ Finishing touches

• 📝 Generate docstrings

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}