Summary: | custom http user-agent in check-url.pl (fix for books.google.com 401 error) | ||
---|---|---|---|
Product: | Koha | Reporter: | Judit <judit> |
Component: | Command-line Utilities | Assignee: | Galen Charlton <gmcharlt> |
Status: | CLOSED FIXED | QA Contact: | Marcel de Rooy <m.de.rooy> |
Severity: | trivial | ||
Priority: | P5 - low | CC: | chris, f.demians, jonathan.druart, m.de.rooy, mjr, paul.poulain |
Version: | 3.6 | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | --- | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: | Circulation function: | ||
Bug Depends on: | |||
Bug Blocks: | 7963 | ||
Attachments: |
Bug 6267 – custom http user-agent in check-url.pl (fix for books.google.com 401 error)
Bug 6267 custom http user-agent in check-url.pl (fix for books.google.com 401 error) Fix a typo Bug 6267: [SIGNED-OFF]Fix a typo |
Description
Judit
2011-04-26 01:43:24 UTC
Created attachment 8913 [details] [review] Bug 6267 – custom http user-agent in check-url.pl (fix for books.google.com 401 error) Patch by Judit with a small change to the help wording. Sponsored by CALYX information essentials. Created attachment 8937 [details] [review] Bug 6267 custom http user-agent in check-url.pl (fix for books.google.com 401 error) Patch by Judit with a small change to the help wording. Sponsored by CALYX information essentials. Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz> QA Comment: Looks good to me. Did not test it. Patch only touches a cron job, no template changes or translation issues. Passed QA. Due to this patch, check-url.pl script fails with this error: syntax error at ./check-url.pl line 168, near "$agent;" syntax error at ./check-url.pl line 170, near ")" Execution of ./check-url.pl aborted due to compilation errors (#1) See also alternative check URL script: bug 7963. Created attachment 9769 [details] [review] Fix a typo Created attachment 9833 [details] [review] Bug 6267: [SIGNED-OFF]Fix a typo Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> QA comment : tiny & obvious patch replacing a ; by a , to avoid an error Patch pushed on master branch, I can't push it on new/bug_6267 Should be backported in 3.8 Pushed to 3.8.x will be in 3.8.2 |