CONTRIBUTING.md 18 KB
Newer Older
Philipp Hagemeister's avatar
Philipp Hagemeister committed
1
**Please include the full output of youtube-dl when run with `-v`**, i.e. **add** `-v` flag to **your command line**, copy the **whole** output and post it in the issue body wrapped in \`\`\` for better formatting. It should look similar to this:
Philipp Hagemeister's avatar
Philipp Hagemeister committed
2
```
Philipp Hagemeister's avatar
Philipp Hagemeister committed
3
$ youtube-dl -v <your command line>
Philipp Hagemeister's avatar
Philipp Hagemeister committed
4 5 6 7 8 9 10 11 12 13 14
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'http://www.youtube.com/watch?v=BaW_jenozKcj']
[debug] Encodings: locale cp1251, fs mbcs, out cp866, pref cp1251
[debug] youtube-dl version 2015.12.06
[debug] Git HEAD: 135392e
[debug] Python version 2.6.6 - Windows-2003Server-5.2.3790-SP2
[debug] exe versions: ffmpeg N-75573-g1d0487f, ffprobe N-75573-g1d0487f, rtmpdump 2.4
[debug] Proxy map: {}
...
```
Sergey M․'s avatar
Sergey M․ committed
15
**Do not post screenshots of verbose logs; only plain text is acceptable.**
Philipp Hagemeister's avatar
Philipp Hagemeister committed
16

Philipp Hagemeister's avatar
Philipp Hagemeister committed
17
The output (including the first lines) contains important debugging information. Issues without the full output are often not reproducible and therefore do not get solved in short order, if ever.
18 19 20 21 22 23 24 25 26 27 28 29 30

Please re-read your issue once again to avoid a couple of common mistakes (you can and should use this as a checklist):

### Is the description of the issue itself sufficient?

We often get issue reports that we cannot really decipher. While in most cases we eventually get the required information after asking back multiple times, this poses an unnecessary drain on our resources. Many contributors, including myself, are also not native speakers, so we may misread some parts.

So please elaborate on what feature you are requesting, or what bug you want to be fixed. Make sure that it's obvious

- What the problem is
- How it could be fixed
- How your proposed solution would look like

Jakub Wilk's avatar
Jakub Wilk committed
31
If your report is shorter than two lines, it is almost certainly missing some of these, which makes it hard for us to respond to it. We're often too polite to close the issue outright, but the missing info makes misinterpretation likely. As a committer myself, I often get frustrated by these issues, since the only possible way for me to move forward on them is to ask for clarification over and over.
32

Philipp Hagemeister's avatar
Philipp Hagemeister committed
33
For bug reports, this means that your report should contain the *complete* output of youtube-dl when called with the `-v` flag. The error message you get for (most) bugs even says so, but you would not believe how many of our bug reports do not contain this information.
34

Philipp Hagemeister's avatar
Philipp Hagemeister committed
35
If your server has multiple IPs or you suspect censorship, adding `--call-home` may be a good idea to get more diagnostics. If the error is `ERROR: Unable to extract ...` and you cannot reproduce it from multiple countries, add `--dump-pages` (warning: this will yield a rather large output, redirect it to the file `log.txt` by adding `>log.txt 2>&1` to your command-line) or upload the `.dump` files you get when you add `--write-pages` [somewhere](https://gist.github.com/).
Philipp Hagemeister's avatar
Philipp Hagemeister committed
36

Philipp Hagemeister's avatar
Philipp Hagemeister committed
37
**Site support requests must contain an example URL**. An example URL is a URL you might want to download, like `http://www.youtube.com/watch?v=BaW_jenozKc`. There should be an obvious video present. Except under very special circumstances, the main page of a video service (e.g. `http://www.youtube.com/`) is *not* an example URL.
38 39 40

###  Are you using the latest version?

Philipp Hagemeister's avatar
Philipp Hagemeister committed
41
Before reporting any issue, type `youtube-dl -U`. This should report that you're up-to-date. About 20% of the reports we receive are already fixed, but people are using outdated versions. This goes for feature requests as well.
42 43 44

###  Is the issue already documented?

Philipp Hagemeister's avatar
Philipp Hagemeister committed
45
Make sure that someone has not already opened the issue you're trying to open. Search at the top of the window or browse the [GitHub Issues](https://github.com/rg3/youtube-dl/search?type=Issues) of this repository. If there is an issue, feel free to write something along the lines of "This affects me as well, with version 2015.01.01. Here is some more information on the issue: ...". While some issues may be old, a new post into them often spurs rapid activity.
46 47 48

###  Why are existing options not enough?

Sergey M․'s avatar
Sergey M․ committed
49
Before requesting a new feature, please have a quick peek at [the list of supported options](https://github.com/rg3/youtube-dl/blob/master/README.md#options). Many feature requests are for features that actually exist already! Please, absolutely do show off your work in the issue report and detail how the existing similar options do *not* solve your problem.
50 51 52 53 54 55 56 57 58 59 60

###  Is there enough context in your bug report?

People want to solve problems, and often think they do us a favor by breaking down their larger problems (e.g. wanting to skip already downloaded files) to a specific request (e.g. requesting us to look whether the file exists before downloading the info page). However, what often happens is that they break down the problem into two steps: One simple, and one impossible (or extremely complicated one).

We are then presented with a very complicated request when the original problem could be solved far easier, e.g. by recording the downloaded video IDs in a separate file. To avoid this, you must include the greater context where it is non-obvious. In particular, every feature request that does not consist of adding support for a new site should contain a use case scenario that explains in what situation the missing feature would be useful.

###  Does the issue involve one problem, and one problem only?

Some of our users seem to think there is a limit of issues they can or should open. There is no limit of issues they can or should open. While it may seem appealing to be able to dump all your issues into one ticket, that means that someone who solves one of your issues cannot mark the issue as closed. Typically, reporting a bunch of issues leads to the ticket lingering since nobody wants to attack that behemoth, until someone mercifully splits the issue into multiple ones.

Sergey M․'s avatar
Sergey M․ committed
61
In particular, every site support request issue should only pertain to services at one site (generally under a common domain, but always using the same backend technology). Do not request support for vimeo user videos, White house podcasts, and Google Plus pages in the same issue. Also, make sure that you don't post bug reports alongside feature requests. As a rule of thumb, a feature request does not include outputs of youtube-dl that are not immediately related to the feature at hand. Do not post reports of a network error alongside the request for a new video service.
62 63 64

###  Is anyone going to need the feature?

Kieran O'Reilly's avatar
Kieran O'Reilly committed
65
Only post features that you (or an incapacitated friend you can personally talk to) require. Do not post features because they seem like a good idea. If they are really useful, they will be requested by someone who requires them.
66 67 68

###  Is your question about youtube-dl?

Sergey M․'s avatar
Sergey M․ committed
69
It may sound strange, but some bug reports we receive are completely unrelated to youtube-dl and relate to a different, or even the reporter's own, application. Please make sure that you are actually using youtube-dl. If you are using a UI for youtube-dl, report the bug to the maintainer of the actual application providing the UI. On the other hand, if your UI for youtube-dl fails in some way you believe is related to youtube-dl, by all means, go ahead and report the bug.
70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87

# DEVELOPER INSTRUCTIONS

Most users do not need to build youtube-dl and can [download the builds](http://rg3.github.io/youtube-dl/download.html) or get them from their distribution.

To run youtube-dl as a developer, you don't need to build anything either. Simply execute

    python -m youtube_dl

To run the test, simply invoke your favorite test runner, or execute a test file directly; any of the following work:

    python -m unittest discover
    python test/test_download.py
    nosetests

If you want to create a build of youtube-dl yourself, you'll need

* python
Sergey M․'s avatar
Sergey M․ committed
88
* make (only GNU make is supported)
89 90 91 92 93 94
* pandoc
* zip
* nosetests

### Adding support for a new site

95
If you want to add support for a new site, first of all **make sure** this site is **not dedicated to [copyright infringement](README.md#can-you-add-support-for-this-anime-video-site-or-site-which-shows-current-movies-for-free)**. youtube-dl does **not support** such sites thus pull requests adding support for them **will be rejected**.
Philipp Hagemeister's avatar
Philipp Hagemeister committed
96

Sergey M․'s avatar
Sergey M․ committed
97
After you have ensured this site is distributing its content legally, you can follow this quick list (assuming your service is called `yourextractor`):
98 99

1. [Fork this repository](https://github.com/rg3/youtube-dl/fork)
Sergey M․'s avatar
Sergey M․ committed
100 101 102 103 104 105 106 107 108
2. Check out the source code with:

        git clone git@github.com:YOUR_GITHUB_USERNAME/youtube-dl.git

3. Start a new git branch with

        cd youtube-dl
        git checkout -b yourextractor

109
4. Start with this simple template and save it to `youtube_dl/extractor/yourextractor.py`:
Sergey M․'s avatar
Sergey M․ committed
110

111 112 113 114 115 116 117 118 119 120 121 122 123 124 125 126
    ```python
    # coding: utf-8
    from __future__ import unicode_literals

    from .common import InfoExtractor


    class YourExtractorIE(InfoExtractor):
        _VALID_URL = r'https?://(?:www\.)?yourextractor\.com/watch/(?P<id>[0-9]+)'
        _TEST = {
            'url': 'http://yourextractor.com/watch/42',
            'md5': 'TODO: md5 sum of the first 10241 bytes of the video file (use --test)',
            'info_dict': {
                'id': '42',
                'ext': 'mp4',
                'title': 'Video title goes here',
Sergey M․'s avatar
Sergey M․ committed
127
                'thumbnail': r're:^https?://.*\.jpg$',
128 129 130 131 132 133 134 135 136 137 138 139 140
                # TODO more properties, either as:
                # * A value
                # * MD5 checksum; start the string with md5:
                # * A regular expression; start the string with re:
                # * Any Python type (for example int or float)
            }
        }

        def _real_extract(self, url):
            video_id = self._match_id(url)
            webpage = self._download_webpage(url, video_id)

            # TODO more code goes here, for example ...
Philipp Hagemeister's avatar
Philipp Hagemeister committed
141
            title = self._html_search_regex(r'<h1>(.+?)</h1>', webpage, 'title')
142 143 144 145 146

            return {
                'id': video_id,
                'title': title,
                'description': self._og_search_description(webpage),
Philipp Hagemeister's avatar
Philipp Hagemeister committed
147
                'uploader': self._search_regex(r'<div[^>]+id="uploader"[^>]*>([^<]+)<', webpage, 'uploader', fatal=False),
148 149 150
                # TODO more properties (see youtube_dl/extractor/common.py)
            }
    ```
Philipp Hagemeister's avatar
Philipp Hagemeister committed
151
5. Add an import in [`youtube_dl/extractor/extractors.py`](https://github.com/rg3/youtube-dl/blob/master/youtube_dl/extractor/extractors.py).
Philipp Hagemeister's avatar
Philipp Hagemeister committed
152
6. Run `python test/test_download.py TestDownload.test_YourExtractor`. This *should fail* at first, but you can continually re-run it until you're done. If you decide to add more than one test, then rename ``_TEST`` to ``_TESTS`` and make it into a list of dictionaries. The tests will then be named `TestDownload.test_YourExtractor`, `TestDownload.test_YourExtractor_1`, `TestDownload.test_YourExtractor_2`, etc.
Sergey M․'s avatar
Sergey M․ committed
153
7. Have a look at [`youtube_dl/extractor/common.py`](https://github.com/rg3/youtube-dl/blob/master/youtube_dl/extractor/common.py) for possible helper methods and a [detailed description of what your extractor should and may return](https://github.com/rg3/youtube-dl/blob/master/youtube_dl/extractor/common.py#L74-L252). Add tests and code for as many as you want.
Sergey M․'s avatar
Sergey M․ committed
154 155
8. Make sure your code follows [youtube-dl coding conventions](#youtube-dl-coding-conventions) and check the code with [flake8](https://pypi.python.org/pypi/flake8). Also make sure your code works under all [Python](http://www.python.org/) versions claimed supported by youtube-dl, namely 2.6, 2.7, and 3.2+.
9. When the tests pass, [add](http://git-scm.com/docs/git-add) the new files and [commit](http://git-scm.com/docs/git-commit) them and [push](http://git-scm.com/docs/git-push) the result, like this:
156

Philipp Hagemeister's avatar
Philipp Hagemeister committed
157
        $ git add youtube_dl/extractor/extractors.py
158 159 160 161
        $ git add youtube_dl/extractor/yourextractor.py
        $ git commit -m '[yourextractor] Add new extractor'
        $ git push origin yourextractor

Sergey M․'s avatar
Sergey M․ committed
162
10. Finally, [create a pull request](https://help.github.com/articles/creating-a-pull-request). We'll then review and merge it.
163 164 165

In any case, thank you very much for your contributions!

Sergey M․'s avatar
Sergey M․ committed
166 167 168 169
## youtube-dl coding conventions

This section introduces a guide lines for writing idiomatic, robust and future-proof extractor code.

Sergey M․'s avatar
Sergey M․ committed
170
Extractors are very fragile by nature since they depend on the layout of the source data provided by 3rd party media hosters out of your control and this layout tends to change. As an extractor implementer your task is not only to write code that will extract media links and metadata correctly but also to minimize dependency on the source's layout and even to make the code foresee potential future changes and be ready for that. This is important because it will allow the extractor not to break on minor layout changes thus keeping old youtube-dl versions working. Even though this breakage issue is easily fixed by emitting a new version of youtube-dl with a fix incorporated, all the previous versions become broken in all repositories and distros' packages that may not be so prompt in fetching the update from us. Needless to say, some non rolling release distros may never receive an update at all.
Sergey M․'s avatar
Sergey M․ committed
171 172 173

### Mandatory and optional metafields

Sergey M․'s avatar
Sergey M․ committed
174
For extraction to work youtube-dl relies on metadata your extractor extracts and provides to youtube-dl expressed by an [information dictionary](https://github.com/rg3/youtube-dl/blob/master/youtube_dl/extractor/common.py#L75-L257) or simply *info dict*. Only the following meta fields in the *info dict* are considered mandatory for a successful extraction process by youtube-dl:
Sergey M․'s avatar
Sergey M․ committed
175 176 177 178 179

 - `id` (media identifier)
 - `title` (media title)
 - `url` (media download URL) or `formats`

Sergey M․'s avatar
Sergey M․ committed
180
In fact only the last option is technically mandatory (i.e. if you can't figure out the download location of the media the extraction does not make any sense). But by convention youtube-dl also treats `id` and `title` as mandatory. Thus the aforementioned metafields are the critical data that the extraction does not make any sense without and if any of them fail to be extracted then the extractor is considered completely broken.
Sergey M․'s avatar
Sergey M․ committed
181

Sergey M․'s avatar
Sergey M․ committed
182
[Any field](https://github.com/rg3/youtube-dl/blob/master/youtube_dl/extractor/common.py#L149-L257) apart from the aforementioned ones are considered **optional**. That means that extraction should be **tolerant** to situations when sources for these fields can potentially be unavailable (even if they are always available at the moment) and **future-proof** in order not to break the extraction of general purpose mandatory fields.
Sergey M․'s avatar
Sergey M․ committed
183 184 185 186 187 188 189 190 191 192 193 194 195 196 197 198 199 200 201

#### Example

Say you have some source dictionary `meta` that you've fetched as JSON with HTTP request and it has a key `summary`:

```python
meta = self._download_json(url, video_id)
```
    
Assume at this point `meta`'s layout is:

```python
{
    ...
    "summary": "some fancy summary text",
    ...
}
```

Sergey M․'s avatar
Sergey M․ committed
202
Assume you want to extract `summary` and put it into the resulting info dict as `description`. Since `description` is an optional meta field you should be ready that this key may be missing from the `meta` dict, so that you should extract it like:
Sergey M․'s avatar
Sergey M․ committed
203 204 205 206 207 208 209 210 211 212 213

```python
description = meta.get('summary')  # correct
```

and not like:

```python
description = meta['summary']  # incorrect
```

Sergey M․'s avatar
Sergey M․ committed
214
The latter will break extraction process with `KeyError` if `summary` disappears from `meta` at some later time but with the former approach extraction will just go ahead with `description` set to `None` which is perfectly fine (remember `None` is equivalent to the absence of data).
Sergey M․'s avatar
Sergey M․ committed
215 216 217 218 219 220 221 222 223 224 225 226 227 228 229 230 231 232 233

Similarly, you should pass `fatal=False` when extracting optional data from a webpage with `_search_regex`, `_html_search_regex` or similar methods, for instance:

```python
description = self._search_regex(
    r'<span[^>]+id="title"[^>]*>([^<]+)<',
    webpage, 'description', fatal=False)
```

With `fatal` set to `False` if `_search_regex` fails to extract `description` it will emit a warning and continue extraction.

You can also pass `default=<some fallback value>`, for example:

```python
description = self._search_regex(
    r'<span[^>]+id="title"[^>]*>([^<]+)<',
    webpage, 'description', default=None)
```

Sergey M․'s avatar
Sergey M․ committed
234
On failure this code will silently continue the extraction with `description` set to `None`. That is useful for metafields that may or may not be present.
Sergey M․'s avatar
Sergey M․ committed
235 236 237
 
### Provide fallbacks

Sergey M․'s avatar
Sergey M․ committed
238
When extracting metadata try to do so from multiple sources. For example if `title` is present in several places, try extracting from at least some of them. This makes it more future-proof in case some of the sources become unavailable.
Sergey M․'s avatar
Sergey M․ committed
239 240 241

#### Example

Sergey M․'s avatar
Sergey M․ committed
242
Say `meta` from the previous example has a `title` and you are about to extract it. Since `title` is a mandatory meta field you should end up with something like:
Sergey M․'s avatar
Sergey M․ committed
243 244 245 246 247

```python
title = meta['title']
```

Sergey M․'s avatar
Sergey M․ committed
248
If `title` disappears from `meta` in future due to some changes on the hoster's side the extraction would fail since `title` is mandatory. That's expected.
Sergey M․'s avatar
Sergey M․ committed
249 250 251 252 253 254 255 256 257 258 259 260 261 262 263 264 265 266 267 268 269 270 271 272 273 274 275 276 277 278 279 280 281 282 283 284

Assume that you have some another source you can extract `title` from, for example `og:title` HTML meta of a `webpage`. In this case you can provide a fallback scenario:

```python
title = meta.get('title') or self._og_search_title(webpage)
```

This code will try to extract from `meta` first and if it fails it will try extracting `og:title` from a `webpage`.

### Make regular expressions flexible

When using regular expressions try to write them fuzzy and flexible.
 
#### Example

Say you need to extract `title` from the following HTML code:

```html
<span style="position: absolute; left: 910px; width: 90px; float: right; z-index: 9999;" class="title">some fancy title</span>
```

The code for that task should look similar to:

```python
title = self._search_regex(
    r'<span[^>]+class="title"[^>]*>([^<]+)', webpage, 'title')
```

Or even better:

```python
title = self._search_regex(
    r'<span[^>]+class=(["\'])title\1[^>]*>(?P<title>[^<]+)',
    webpage, 'title', group='title')
```

Sergey M․'s avatar
Sergey M․ committed
285
Note how you tolerate potential changes in the `style` attribute's value or switch from using double quotes to single for `class` attribute: 
Sergey M․'s avatar
Sergey M․ committed
286 287 288 289 290 291 292 293 294 295 296 297 298

The code definitely should not look like:

```python
title = self._search_regex(
    r'<span style="position: absolute; left: 910px; width: 90px; float: right; z-index: 9999;" class="title">(.*?)</span>',
    webpage, 'title', group='title')
```

### Use safe conversion functions

Wrap all extracted numeric data into safe functions from `utils`: `int_or_none`, `float_or_none`. Use them for string to number conversions as well.