mafintosh/peerflix

[feature] Read link from stdin

Open

#172 opened on Feb 2, 2015

View on GitHub
 (6 comments) (0 reactions) (0 assignees)JavaScript (6,130 stars) (717 forks)batch import
help wanted

Description

e.g., this usage

echo "magnet://..." | peerflix -l

Motivation: I wrote lukedmor/magnet which retrieves a magnet link. Rather than simply piping the link into peerflix, I have to manually set all peerflix arguments from within my script.

Contributor guide

Tech stack
javascriptnodejsshell
Domain
cli
Issue type
feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
half day
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
Node.js basicsCLI argument parsingstdin stream handling
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
75
Research direction
Study the existing CLI argument parsing in the peerflix repository (likely in the main entry file such as index.js or bin/cli.js). Add a new option ' l' that reads from stdin when no magnet link is provided via command line. Modify the startup flow to check if ' l' is set, then read the link from process.stdin and proceed with streaming. Look at how the torrent link is currently processed to integrate the stdin input appropriately.