Hello, could you please look into it why this feed is considered inv

I agree <a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-

Failed to detect feed type:http://www.dailytelegraph.com.au/feed/ about gofeed HOT 2 CLOSED

meszaroszoltan commented on June 15, 2024

Failed to detect feed type:http://www.dailytelegraph.com.au/feed/

from gofeed.

Comments (2)

Necoro commented on June 15, 2024

That one was tricky:
wget -O - https://www.dailytelegraph.com.au/feed outputs the expected feed.
But when looking at all the redirects:

--2021-06-15 00:12:20--  https://www.dailytelegraph.com.au/feed/
Loaded CA certificate '/etc/ssl/certs/ca-certificates.crt'
Resolving www.dailytelegraph.com.au (www.dailytelegraph.com.au)... 23.3.64.134
Connecting to www.dailytelegraph.com.au (www.dailytelegraph.com.au)|23.3.64.134|:443... connected.
HTTP request sent, awaiting response... 302 Moved Temporarily
Location: https://www.dailytelegraph.com.au/remote/check_cookie.html?url=https%3a%2f%2fwww.dailytelegraph.com.au%2ffeed%2f [following]
--2021-06-15 00:12:20--  https://www.dailytelegraph.com.au/remote/check_cookie.html?url=https%3a%2f%2fwww.dailytelegraph.com.au%2ffeed%2f
Reusing existing connection to www.dailytelegraph.com.au:443.
HTTP request sent, awaiting response... 302 Moved Temporarily
Location: https://www.dailytelegraph.com.au/feed/ [following]

one gets the idea: Cookies.

And thus:
wget --no-cookies -O - https://www.dailytelegraph.com.au/feed/ returns an HTML site that states that cookies need to be enabled.

Alternatives:

As cookies and RSS in combination make no sense, one should contact the Daily Telegraph and make them correct their broken implementation
Make gofeed support cookies: In parser.go#ParseURLWithContext:

if client.Jar, err = cookiejar.New(nil); err != nil {
	return
}

Use your own http.Client and call parse.Parse with the result.

@mmcdole: How do you feel about this?

from gofeed.

mmcdole commented on June 15, 2024

I agree @Necoro -- this is probably outside of what I'd consider normal. For this, I'd do as you suggested and have them create their own http.Client with this behavior.

from gofeed.

Recommend Projects