[koha-commits] main Koha release repository branch master updated. v16.05.00-438-ge4bb70a

[Git repo owner]

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  e4bb70a447c7a6ea5e7498250adbe0e152259974 (commit)
       via  6e38b8b60f118c0721a52fdaa332446063657dd3 (commit)
      from  30474a321575ffecfd4e2578dce59385267ec8d7 (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 e4bb70a447c7a6ea5e7498250adbe0e152259974
Author: Tomas Cohen Arazi <tomascohen at theke.io>
Date:   Thu Aug 4 11:47:16 2016 -0300

    Bug 17030: Make REST api available on packages with plack enabled
    
    This patch is the starting point for making the REST api available
    on Plack.
    
    What it does:
    - It creates the /api/v1/app.pl mount point in plack.psgi
    - It enables the ProxyPass and ProxyPassReverse directives so it
      is reached through Plack.
    - It sets rewrite rules so we can use the 'pretty' urls (i.e.
      /api/v1/patrons instead of /api/v1/app.pl/api/v1/patrons).
    
    To test:
    - Grab the following files, and put them in /etc/koha (overwrite the existing ones)
      debian/templates/apache-shared-intranet-plack.conf
      debian/templates/apache-shared-opac-plack.conf
    - Tweak your /etc/koha/sites/kohadev/plack.psgi file so the API-related stuff
      is present on your file.
    - Make sure Plack is enabled for the instance:
      $ sudo koha-plack --enable kohadev
      $ sudo koha-plack --restart kohadev
      $ sudo service apache2 restart
    - Follow the previous patch test plan, but use this URLs (no pretty URL):
    
      http://localhost:8080/api/v1/app.pl/api/v1/patrons/50
      http://localhost:8081/api/v1/app.pl/api/v1/patrons/50
    => SUCCESS: You get a JSON response from the API [1]
    - Not use this URLs:
      http://localhost:8080/api/v1/patrons/50
      http://localhost:8081/api/v1/patrons/50
    => SUCCESS: You get a JSON response from the API [1]
    - Sign off :-D
    
    [1] this patch made a bug visible (the session is lost when accessing the API through
    Plack) but it shouldn't prevent its inclusion because the API right now is not even available
    as default for developers to test or fix it.
    
    Signed-off-by: Benjamin Rokseth <benjamin.rokseth at kul.oslo.kommune.no>
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>

commit 6e38b8b60f118c0721a52fdaa332446063657dd3
Author: Tomas Cohen Arazi <tomascohen at theke.io>
Date:   Thu Aug 4 11:39:25 2016 -0300

    Bug 17030: Enable REST api on packages
    
    This patch enables access to the REST API endpoint on packages setup.
    It does so, by patching the shared apache-shared-intranet.conf and
    apache-shared-opac.conf.
    
    You can build your own master packages with this patch applied, or just:
    
    - Grab
      debian/templates/apache-shared-intranet.conf
      debian/templates/apache-shared-opac.conf
    and overwrite their counterparts in /etc/koha on a packages setup. For example
    in kohadevbox.
    - Have Koha loaded with all default data
    - Create a superlibrarian user for you
    - Login to the intranet and the OPAC
    - Point your browser to:
      http://localhost:8080/api/v1/patrons/51
    => SUCCESS: You get JSON data, for the patron you requested
      http://localhost:8081/api/v1/patrons/51
    => SUCCESS: You get JSON data, for the patron you requested
    - Sign off :-D
    
    Note: I use the HTTPRequester addon for Firefox, re-using the CGISESSID value from the
    browser session cookie, in the headers.
    
    Signed-off-by: Benjamin Rokseth <benjamin.rokseth at kul.oslo.kommune.no>
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>

-----------------------------------------------------------------------

Summary of changes:
 debian/templates/apache-shared-intranet-plack.conf |    7 +++++--
 debian/templates/apache-shared-intranet.conf       |   14 ++++++++++++++
 debian/templates/apache-shared-opac-plack.conf     |    7 +++++--
 debian/templates/apache-shared-opac.conf           |   14 ++++++++++++++
 debian/templates/plack.psgi                        |   16 ++++++++++------
 5 files changed, 48 insertions(+), 10 deletions(-)


hooks/post-receive
-- 
main Koha release repository