hadley/rvest — pull request #37 by rappster (2014-12-10 20:28:46+00:00)
Title: Additional args to be passed to `httr::GET()`

Hi Hadley,

this is my first pull request - so please don't bite ;-)

## Reason for modification
I needed to pass along altered user agent information for the underlying `GET` request in `html()`

## Actual modification
In order to avoid this

```
require("httr")
content(GET(, user_agent("Mozilla/5.0")))
```

I modified `html()`: it now accepts `...` and passes them along to `httr::GET()` in order for this to work

```
html(, user_agent("Mozilla/5.0")
```

As this involves the heed to explicitly specify `encoding = NULL` in calls to `html()` when wanting to pass additional things along (unless making `...` the second argument in `html()`), you might want to go with the second (commented-out) implementation of `html()` featuring and actual formal argument `httr_config` for the things to pass to `httr::GET()`.

## Unit test
I also added a short unit test for a German website that throws a `403` error for the default user agent.

Best regards and thanks for the cool package!