xmltv icon indicating copy to clipboard operation
xmltv copied to clipboard

Support Schedules Direct redirection response

Open garybuhrmaster opened this issue 2 years ago • 1 comments

The new Schedules Direct JSON endpoint may issue a redirect to a S3 bucket. Allow LWP to follow redirects for additional methods.

Thanks for taking the time to open a Pull Request (PR). Please take a moment to review our open/closed PRs above, in case a similar contribution has already been offered.

If you are opening a new Pull Request, please give it a descriptive title and fill out the blanks below, providing as much information as possible.

Pull Requests should be made against our master branch, and rebased if necessary.

What type of Pull Request is this?

  • [ ] adds new functionality
  • [X] fixes/improves existing functionality

Does this PR close any currently open issues?

(If yes, please provide the issue number) …

Please explain what this PR does

This fixes a fetch issue with the new Schedules Direct JSON endpoint.

Any other information?

(For example, has this change been discussed on the xmltv-devel mailing list?) …

Where have you tested these changes?

Operating System:

Linux

Perl Version:

5.34

garybuhrmaster avatar May 21 '22 11:05 garybuhrmaster

As mentioned elsewhere, this patch is necessary, but not sufficient (there are other issues in the zz_sdjson grabber). Hopefully an interested party will review the code and propose additional additional fixes.

garybuhrmaster avatar May 22 '22 15:05 garybuhrmaster