Error redirection forbidden

[marceloboeira]

Whenever the feed's link redirects us a RuntimeError is generated.

>> PodcastReader.new("http://canaltech.com.br/rss/podcast/")
RuntimeError: redirection forbidden: http://canaltech.com.br/rss/podcast/ -> https://canaltech.com.br/rss/podcast/
>>

[kule]

Thanks for this Marcelo, I'll have a look tomorrow - couple of things with a quick look:

1. The current test doesn't seem to fail if I remove the added redirection code?
2. I was wondering if we could do without the extra dependency and just use something like the redirect behaviour shown here instead: https://twin.github.io/improving-open-uri/

[marceloboeira]

@kule

1. I will have to investigate if ShamRack is redirecting for real or if the problem was with a specific kind of redirection.

Update The problems seems to be the http->https redirection, still investigating.

2. I looked at it, but it seemed quite an unnecessary workaround.

OpenURI itself does not seem an appropriate dependency, it would be interesting to have an injected dependency to handle the XML download, that would be my next step. I was handling the redirection with the dependency temporarily.