Attachment 151124 Details for Bug 32995 – Bug 32995: Pass user agent string for OverDrive fulfillment

[patch] Bug 32995: Pass user agent string for OverDrive fulfillment

Bug-32995-Pass-user-agent-string-for-OverDrive-ful.patch (text/plain), 1.53 KB, created by Marcel de Rooy on 2023-05-12 09:14:02 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Marcel de Rooy

Created: 2023-05-12 09:14:02 UTC

Size: 1.53 KB

patch

obsolete

>From 0e007f908d3955800f5519539568d9c888a78783 Mon Sep 17 00:00:00 2001
>From: Nick Clemens <nick@bywatersolutions.com>
>Date: Fri, 17 Feb 2023 13:01:02 +0000
>Subject: [PATCH] Bug 32995: Pass user agent string for OverDrive fulfillment
>Content-Type: text/plain; charset=utf-8
>
>This patch simply adds the string Koha/VERSION to the fulfillment
>request
>
>TO test:
>1 - Have OverDrive credentials for testing
>2 - Enable all the overdrive options in system preferences
>3 - Sign in to opac with an overdrive user and verify you can checkout
>  and download titles
>4 - Sign off
>
>Signed-off-by: Magnus Enger <magnus@libriotech.no>
>Not tested with OverDrive, but change looks harmless enough.
>
>Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>---
> Koha/ExternalContent/OverDrive.pm | 2 ++
> 1 file changed, 2 insertions(+)
>
>diff --git a/Koha/ExternalContent/OverDrive.pm b/Koha/ExternalContent/OverDrive.pm
>index 6c6ccf1ae8..bf355a2d4a 100644
>--- a/Koha/ExternalContent/OverDrive.pm
>+++ b/Koha/ExternalContent/OverDrive.pm
>@@ -23,6 +23,7 @@ use Carp qw( croak );
> use base qw(Koha::ExternalContent);
> use WebService::ILS::OverDrive::Patron;
> use C4::Context;
>+use Koha;
> use LWP::UserAgent;
> 
> =head1 NAME
>@@ -223,6 +224,7 @@ sub checkout_download_url {
> 
>     my $ua = LWP::UserAgent->new;
>     $ua->max_redirect(0);
>+    $ua->agent( 'Koha/'.Koha::version() );
>     my $response = $ua->get(
>         "https://patron.api.overdrive.com/v1/patrons/me/checkouts/".$item_id."/formats/downloadredirect",
>         'Authorization' => "Bearer ".$self->client->access_token,
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 32995: 146841 | 150182 | 151124