[koha-commits] main Koha release repository branch master updated. v17.05.00-644-gb4e525e
Git repo owner
gitmaster at git.koha-community.org
Thu Sep 21 17:52:55 CEST 2017
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "main Koha release repository".
The branch, master has been updated
via b4e525e4f6382a421cb7e77632fca8ecaaf8f3d6 (commit)
via 256c66752f2817892b03938edb375b234a616fcd (commit)
via 2bf98402a2d98461b638a584413184b57beaee17 (commit)
via 47b6026c6a3d57cfc86d90fcb72c0d68266b214c (commit)
via fbda925f85ec96651483bd51efed5853b34e8dfb (commit)
via 6c93ea0be10080817cc84a75f5e99c69655d012e (commit)
via 2918cc9c2712d5b658a207b15dcd63718b8f952e (commit)
via 1f9e262ab5d3ac8aa95c771e236693135a2a2c1b (commit)
via 047508bb8d6f3cedfe50d7be360972cd8df3d68a (commit)
via b007919e35c7588b0d95c5257b5334e26a38300b (commit)
via 915963ba7f1580f8d7aaa0b5563c974b7254874a (commit)
via e2fc5247012a99b4242370248a061681d84174f2 (commit)
via ea6c6f3026adce1f0eecf58752f7ae01494d60ed (commit)
from 164b13690ab133b499ebeb2a809c8c8649debbfe (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
- Log -----------------------------------------------------------------
commit b4e525e4f6382a421cb7e77632fca8ecaaf8f3d6
Author: Lari Taskula <lari.taskula at jns.fi>
Date: Thu Mar 16 13:53:44 2017 +0200
Bug 18282: operationId must be unique
operationId has the following documentation:
"Unique string used to identify the operation. The id MUST be unique among all
operations described in the API."
This patch modifies operationIds to be unique accross our API operations.
Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>
Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
commit 256c66752f2817892b03938edb375b234a616fcd
Author: Olli-Antti Kivilahti <olli-antti.kivilahti at jns.fi>
Date: Fri Mar 17 08:09:05 2017 +0200
Bug 18290: Fix t/db_dependent/Koha/Object.t, Mojo::JSON::Bool is a JSON::PP::Boolean :)
Mojolicious 7.21 onwards, no longer returns Mojo::JSON::Bool-objects but JSON::PP instead.
Which might be pretty smart.
This version is required by bug 18137 and so this patch for the tests is
needed.
Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>
Tested along with 18137 and its dependencies (libs).
Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
commit 2bf98402a2d98461b638a584413184b57beaee17
Author: Tomas Cohen Arazi <tomascohen at theke.io>
Date: Wed Aug 9 11:11:13 2017 -0300
Bug 18137: (QA followup) Make sure the session exists and is expired on expiration tests
Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>
Signed-off-by: Lari Taskula <lari.taskula at jns.fi>
Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
commit 47b6026c6a3d57cfc86d90fcb72c0d68266b214c
Author: Lari Taskula <lari.taskula at jns.fi>
Date: Fri Feb 17 13:14:09 2017 +0200
Bug 18137: List Mojolicious::Plugin::OpenAPI and JSON::Validator as dependencies
Edit (tcohen): I've changed the version numbers to match those Mirko has already
successfully packaged and are known to work for this patchset.
Signed-off-by: Olli-Antti Kivilahti <olli-antti.kivilahti at jns.fi>
Signed-off-by: Josef Moravec <josef.moravec at gmail.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>
Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
commit fbda925f85ec96651483bd51efed5853b34e8dfb
Author: Lari Taskula <lari.taskula at jns.fi>
Date: Thu Mar 16 14:24:52 2017 +0200
Bug 18137: (follow-up) Sort definitions.json
Before this file grows, we should sort it alphabetically.
To test:
1. prove t/db_dependent/api/v1
Signed-off-by: Josef Moravec <josef.moravec at gmail.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>
Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
commit 6c93ea0be10080817cc84a75f5e99c69655d012e
Author: Lari Taskula <lari.taskula at jns.fi>
Date: Tue Mar 7 13:31:04 2017 +0200
Bug 18137: (QA-follow-up) Fix pod fail
Signed-off-by: Josef Moravec <josef.moravec at gmail.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>
Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
commit 2918cc9c2712d5b658a207b15dcd63718b8f952e
Author: Lari Taskula <lari.taskula at jns.fi>
Date: Tue Mar 7 13:37:09 2017 +0200
Bug 18137: (follow-up) Fix handling DBIx::Class::Exception messages
- DBIx::Class::Exception should use ->{msg}
Signed-off-by: Josef Moravec <josef.moravec at gmail.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>
Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
commit 1f9e262ab5d3ac8aa95c771e236693135a2a2c1b
Author: Lari Taskula <lari.taskula at jns.fi>
Date: Tue Feb 21 19:55:28 2017 +0200
Bug 18137: Remove x-mojo-around-action
Mojolicious::Plugin::OpenAPI does not support x-mojo-around action. This patch
removes it from our specification document.
Signed-off-by: Olli-Antti Kivilahti <olli-antti.kivilahti at jns.fi>
Signed-off-by: Josef Moravec <josef.moravec at gmail.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>
Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
commit 047508bb8d6f3cedfe50d7be360972cd8df3d68a
Author: Lari Taskula <lari.taskula at jns.fi>
Date: Mon Feb 20 20:07:26 2017 +0200
Bug 18137: Make /holds Mojolicious::Plugin::OpenAPI compatible
Also
- adding some missing and new response definitions into Swagger spec.
- fixing failing tests due to Bug 17932's change of boolean values
To test:
1. prove t/db_dependent/api/v1/holds.t
Signed-off-by: Olli-Antti Kivilahti <olli-antti.kivilahti at jns.fi>
Signed-off-by: Josef Moravec <josef.moravec at gmail.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>
Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
commit b007919e35c7588b0d95c5257b5334e26a38300b
Author: Lari Taskula <lari.taskula at jns.fi>
Date: Mon Feb 20 19:58:28 2017 +0200
Bug 18137: Make /cities Mojolicious::Plugin::OpenAPI compatible
Also:
- adding some missing and new response definitions into Swagger spec.
To test:
1. prove t/db_dependent/api/v1/cities.t
Signed-off-by: Olli-Antti Kivilahti <olli-antti.kivilahti at jns.fi>
Signed-off-by: Josef Moravec <josef.moravec at gmail.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>
Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
commit 915963ba7f1580f8d7aaa0b5563c974b7254874a
Author: Lari Taskula <lari.taskula at jns.fi>
Date: Fri Feb 17 14:59:24 2017 +0200
Bug 18137: Make /patrons Mojolicious::Plugin::OpenAPI compatible
Also:
- adding some missing and new response definitions into Swagger spec.
- fixing failing test due to Bug 17932's change of boolean values
To test:
1. prove t/db_dependent/api/v1/patrons.t
Signed-off-by: Olli-Antti Kivilahti <olli-antti.kivilahti at jns.fi>
Signed-off-by: Josef Moravec <josef.moravec at gmail.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>
Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
commit e2fc5247012a99b4242370248a061681d84174f2
Author: Lari Taskula <lari.taskula at jns.fi>
Date: Fri Feb 17 14:36:36 2017 +0200
Bug 18137: Migrate from Swagger2 to Mojolicious::Plugin::OpenAPI
This patch migrates from Swagger2 to Mojolicious::Plugin::OpenAPI as Swagger2 is
no longer actively maintained.
This migration involves some minor changes to our Swagger specification documents
and to controllers. Each operation is migrated in following patches separately.
Please see Mojolicious::Plugin::OpenAPI and its tutorial for more documentation.
The patch also refactors some API authentication -related code by taking advantage
of Koha::Exceptions. Authentication is now handled via Mojolicious's "under->to"
functionality. The actual authentication & authorization checks are moved to
Koha::REST::V1::Auth. Added a HTTP 503 response for when database update is
required, instead of returning an authentication failure as before.
To test:
1. prove t/db_dependent/api/v1/auth.t
Signed-off-by: Olli-Antti Kivilahti <olli-antti.kivilahti at jns.fi>
Signed-off-by: Josef Moravec <josef.moravec at gmail.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>
Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
commit ea6c6f3026adce1f0eecf58752f7ae01494d60ed
Author: Lari Taskula <lari.taskula at jns.fi>
Date: Fri Feb 17 14:34:42 2017 +0200
Bug 18137: Add useful Koha::Exceptions
Signed-off-by: Olli-Antti Kivilahti <olli-antti.kivilahti at jns.fi>
Signed-off-by: Josef Moravec <josef.moravec at gmail.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>
Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
-----------------------------------------------------------------------
Summary of changes:
C4/Installer/PerlDependencies.pm | 11 +-
Koha/Exceptions.pm | 9 ++
Koha/Exceptions/Authentication.pm | 21 ++++
Koha/Exceptions/Authorization.pm | 18 +++
Koha/REST/V1.pm | 248 +------------------------------------
Koha/REST/{V1.pm => V1/Auth.pm} | 159 +++++++++++++++---------
Koha/REST/V1/Cities.pm | 71 ++++++-----
Koha/REST/V1/Hold.pm | 59 +++++----
Koha/REST/V1/Patron.pm | 17 ++-
api/v1/swagger/definitions.json | 14 +--
api/v1/swagger/paths/cities.json | 74 ++++++++---
api/v1/swagger/paths/holds.json | 90 +++++++++++++-
api/v1/swagger/paths/patrons.json | 38 ++++++
api/v1/swagger/swagger.json | 1 -
t/db_dependent/Koha/Object.t | 4 +-
t/db_dependent/api/v1/auth.t | 116 +++++++++++++++++
t/db_dependent/api/v1/cities.t | 2 +-
t/db_dependent/api/v1/patrons.t | 2 +-
18 files changed, 555 insertions(+), 399 deletions(-)
create mode 100644 Koha/Exceptions/Authentication.pm
create mode 100644 Koha/Exceptions/Authorization.pm
copy Koha/REST/{V1.pm => V1/Auth.pm} (65%)
create mode 100644 t/db_dependent/api/v1/auth.t
hooks/post-receive
--
main Koha release repository
More information about the koha-commits
mailing list