Bug 19043

Summary: Z39.50 target clio-db.cc.columbia.edu:7090 is no longer publicly available.
Product: Koha Reporter: Barton Chittenden <barton>
Component: Z39.50 / SRU / OpenSearch ServersAssignee: Josef Moravec <josef.moravec>
Status: CLOSED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: trivial    
Priority: P5 - low CC: jonathan.druart, josef.moravec, katrin.fischer, m.de.rooy
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 19043: Z39.50 target clio-db.cc.columbia.edu:7090 is no longer publicly available
[SIGNED OFF] Bug 19043: Z39.50 target clio-db.cc.columbia.edu:7090 is no longer publicly available
Bug 19043: Z39.50 target clio-db.cc.columbia.edu:7090 is no longer publicly available
Bug 19043: (QA follow-up) Add dbrev and print warning

Description Barton Chittenden 2017-08-04 20:03:12 UTC
Per http://library.columbia.edu/find/clio/legacy-help/CLIOLegacy-Z3950.html,

"Z39.50 users: Access to CLIO via the Z39.50 protocol is now restricted to on-campus use. For information about using Z39.50 to search CLIO, please see: http://library.columbia.edu/find/clio/classic-help/z3950.html"

This target is one of the sample servers loaded in community koha, it should be removed from the list of z39.50 servers in the database.
Comment 1 Josef Moravec 2017-08-08 14:34:30 UTC
Created attachment 65641 [details] [review]
Bug 19043: Z39.50 target clio-db.cc.columbia.edu:7090 is no longer publicly available

Test plan:
0) Apply the patch
1) Install all affected languages: de-DE, es-ES, fr-CA and pl-PL
2) Remove and recreate Koha database
3) Go through web installer in English and don't forgot to load sample Z39.50 servers
4) Go to Administration -> Z39.50/SRU Servers and confirm the sample
servers are here, but columbia.edu is not here
5) Repeat 2-4 for other languages: de-DE, es-ES, fr-CA and pl-PL
Comment 2 Katrin Fischer 2017-08-08 21:52:51 UTC
Created attachment 65692 [details] [review]
[SIGNED OFF] Bug 19043: Z39.50 target clio-db.cc.columbia.edu:7090 is no longer publicly available

Test plan:
0) Apply the patch
1) Install all affected languages: de-DE, es-ES, fr-CA and pl-PL
2) Remove and recreate Koha database
3) Go through web installer in English and don't forgot to load sample Z39.50 servers
4) Go to Administration -> Z39.50/SRU Servers and confirm the sample
servers are here, but columbia.edu is not here
5) Repeat 2-4 for other languages: de-DE, es-ES, fr-CA and pl-PL

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 3 Jonathan Druart 2017-08-09 17:38:46 UTC
Should we display a message on the about page and/or during the update database process?
Comment 4 Barton Chittenden 2017-08-09 17:53:15 UTC
(In reply to Jonathan Druart from comment #3)
> Should we display a message on the about page and/or during the update
> database process?

I would say both. More communication is generally better.

Verbosely,

--Barton
Comment 5 Josef Moravec 2017-08-09 18:13:46 UTC
(In reply to Jonathan Druart from comment #3)
> Should we display a message on the about page and/or during the update
> database process?

Good idea, I'll try to implement it, so changing status to assigned.
Comment 6 Katrin Fischer 2017-08-09 18:55:44 UTC
Hm, I think the about page is really getting slow. Maybe an updatedatabase entry is enough in this case as it's not structural issue.
Comment 7 Jonathan Druart 2017-08-09 19:22:46 UTC
(In reply to Katrin Fischer from comment #6)
> Hm, I think the about page is really getting slow. Maybe an updatedatabase
> entry is enough in this case as it's not structural issue.

I am not sure this argument is relevant. You are not supposed to hit this page everyday.
But in this case it may be not necessary to add this check.
Comment 8 Marcel de Rooy 2017-10-09 10:55:41 UTC
Created attachment 67837 [details] [review]
Bug 19043: Z39.50 target clio-db.cc.columbia.edu:7090 is no longer publicly available

Test plan:
0) Apply the patch
1) Install all affected languages: de-DE, es-ES, fr-CA and pl-PL
2) Remove and recreate Koha database
3) Go through web installer in English and don't forgot to load sample Z39.50 servers
4) Go to Administration -> Z39.50/SRU Servers and confirm the sample
servers are here, but columbia.edu is not here
5) Repeat 2-4 for other languages: de-DE, es-ES, fr-CA and pl-PL

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 9 Marcel de Rooy 2017-10-09 10:55:45 UTC
Created attachment 67838 [details] [review]
Bug 19043: (QA follow-up) Add dbrev and print warning

This follow-up takes care of the requested warning at upgrade time.
At the same time it removes the obsolete Columbia record when host and
port are equal to the formerly used values. Note that the new host for
on-campus use has been changed.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 10 Jonathan Druart 2017-10-09 19:18:32 UTC
Pushed to master for 17.11, thanks to everybody involved!