Bug 40163

Summary: Several http links should be moved to https
Product: Koha Reporter: Tomás Cohen Arazi (tcohen) <tomascohen>
Component: Architecture, internals, and plumbingAssignee: Tomás Cohen Arazi (tcohen) <tomascohen>
Status: Pushed to main --- QA Contact: Martin Renvoize (ashimema) <martin.renvoize>
Severity: normal    
Priority: P5 - low CC: blawlor, david, dcook, lucas, martin.renvoize, tomascohen
Version: Main   
Hardware: All   
OS: All   
GIT URL: Change sponsored?: ---
Patch complexity: String patch Documentation contact:
Documentation submission: Text to go in the release notes:
Version(s) released in:
25.11.00
Circulation function:
Bug Depends on:    
Bug Blocks: 40872    
Attachments: Bug 40163: Fix several http addresses in the code docs
Bug 40163: Fix several http addresses in the code docs
Bug 40163: (follow-up) Fix Bugzilla URL
Bug 40163: Fix several http addresses in the code docs
Bug 40163: (follow-up) Fix Bugzilla URL
Bug 40163: Fix several http addresses in the code docs
Bug 40163: (follow-up) Fix Bugzilla URL
Bug 40163: Fix several http addresses in the code docs
Bug 40163: (follow-up) Fix Bugzilla URL
Bug 40163: (follow-up) Fix wrong GNU links and update test

Description Tomás Cohen Arazi (tcohen) 2025-06-18 14:17:42 UTC
FSF and GNU links should be moved to HTTPS.
Comment 1 Tomás Cohen Arazi (tcohen) 2025-06-18 14:44:30 UTC
Created attachment 183333 [details] [review]
Bug 40163: Fix several http addresses in the code docs

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Comment 2 David Nind 2025-06-19 00:12:14 UTC
I've failed QA as for all the release notes in diff, the existing https was changed to http.

A couple of other things noted, but outside the scope of the bug:

URLs changed, excluding release notes:
- http://www.gnu.org/licenses -> https://www.gnu.org/licenses
- http://wiki.koha-community.org/wiki/Shibboleth_Configuration -> https://wiki.koha-community.org/wiki/Shibboleth_Configuration
- http://koha-community.org/ -> https://koha-community.org/
- http://bugs.koha-community.org/ -> https://bugs.koha-community.org/
- http://fsf.org/ -> https://fsf.org/
- http://www.gnu.org/licenses/gpl.html -> https://www.gnu.org/licenses/gpl.html
- http://bugs.koha-community.org/bugzilla3/ -> https://bugs.koha-community.org/bugzilla3/
- http://wiki.koha-community.org/wiki/Bug7284_authority_matching_improvement -> https://wiki.koha-community.org/wiki/Bug7284_authority_matching_improvement
- http://wiki.koha-community.org/wiki/Age_restriction -> https://wiki.koha-community.org/wiki/Age_restriction
- http://wiki.koha-community.org/wiki/OpacHiddenItems -> https://wiki.koha-community.org/wiki/OpacHiddenItems
- http://wiki.koha-community.org/wiki/SQL_Reports_Library -> https://wiki.koha-community.org/wiki/SQL_Reports_Library
- http://wiki.koha-community.org/wiki/Opac_browse_feature -> https://wiki.koha-community.org/wiki/Opac_browse_feature
- http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=2883 -> https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=2883
- http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11078 -> https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11078
- http://wiki.koha-community.org/wiki/Koha_offline_circulation_file_format -> https://wiki.koha-community.org/wiki/Koha_offline_circulation_file_format

Notes: 
- https://fsf.org/ redirects to https://www.fsf.org/
- http://www.gnu.org/licenses/gpl.html redirects to https://www.gnu.org/licenses/gpl-3.0.html
- http://www.w3.org/2001/XMLSchema-instance redirects to https://www.w3.org/2001/XMLSchema-instance 
  . koha-tmpl/intranet-tmpl/prog/en/data/marc21_field_006.xml
  . koha-tmpl/intranet-tmpl/prog/en/data/marc21_field_007.xml
  . koha-tmpl/intranet-tmpl/prog/en/data/marc21_field_008.xml

Should https://bugs.koha-community.org/bugzilla3/ be just https://bugs.koha-community.org/ ?
- debian/docs/koha-common.xml
- debian/docs/koha-create.xml

Says licensed under the General Public License, v2 (originally from 2002):
- admin/aqbudgetperiods.pl
- admin/aqplan.pl
- admin/systempreferences.pl
- authorities/ysearch.pl
- cataloguing/ysearch.pl
- tags/review.pl 

https changed to http for all release notes in the diff:
- misc/release_notes/release_notes_17_05_00.html and misc/release_notes/release_notes_17_05_00.md
- misc/release_notes/release_notes_18_05_00.html and misc/release_notes/release_notes_18_05_00.md
- misc/release_notes/release_notes_20_11_00.html and misc/release_notes/release_notes_20_11_00.md: 
- misc/release_notes/release_notes_21_05_00.html and misc/release_notes/release_notes_21_05_00.md:
- misc/release_notes/release_notes_21_11_00.html and misc/release_notes/release_notes_21_11_00.md
- misc/release_notes/release_notes_22_05_00.html and misc/release_notes/release_notes_22_05_00.md
- misc/release_notes/release_notes_22_11_00.html and misc/release_notes/release_notes_22_11_00.md
- misc/release_notes/release_notes_23_05_00.md
- misc/release_notes/release_notes_23_11_00.md
- misc/release_notes/release_notes_24_05_00.md
- misc/release_notes/release_notes_24_11_00.md
- misc/release_notes/release_notes_25_05_00.md
Comment 3 Tomás Cohen Arazi (tcohen) 2025-09-17 18:22:27 UTC Comment hidden (obsolete)
Comment 4 Tomás Cohen Arazi (tcohen) 2025-09-17 18:30:35 UTC Comment hidden (obsolete)
Comment 5 Tomás Cohen Arazi (tcohen) 2025-09-17 18:33:54 UTC
Created attachment 186547 [details] [review]
Bug 40163: Fix several http addresses in the code docs

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Comment 6 Tomás Cohen Arazi (tcohen) 2025-09-17 18:33:57 UTC
Created attachment 186548 [details] [review]
Bug 40163: (follow-up) Fix Bugzilla URL
Comment 7 Tomás Cohen Arazi (tcohen) 2025-09-17 18:34:36 UTC
I decided not to fix the release notes. It is just too much and probably shouldn't even belong to this repository.
Comment 8 Brendan Lawlor 2025-09-19 13:08:11 UTC
Created attachment 186623 [details] [review]
Bug 40163: Fix several http addresses in the code docs

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Signed-off-by: Brendan Lawlor <blawlor@clamsnet.org>
Comment 9 Brendan Lawlor 2025-09-19 13:08:14 UTC
Created attachment 186624 [details] [review]
Bug 40163: (follow-up) Fix Bugzilla URL

Signed-off-by: Brendan Lawlor <blawlor@clamsnet.org>
Comment 10 Brendan Lawlor 2025-09-19 13:09:35 UTC
I rebased this when I signed off because /App/Plugin/CGIBinKoha.pm had been deleted.
Comment 11 Martin Renvoize (ashimema) 2025-09-22 21:04:26 UTC
Created attachment 186762 [details] [review]
Bug 40163: Fix several http addresses in the code docs

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Signed-off-by: Brendan Lawlor <blawlor@clamsnet.org>
Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk>
Comment 12 Martin Renvoize (ashimema) 2025-09-22 21:04:28 UTC
Created attachment 186763 [details] [review]
Bug 40163: (follow-up) Fix Bugzilla URL

Signed-off-by: Brendan Lawlor <blawlor@clamsnet.org>
Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk>
Comment 13 Lucas Gass (lukeg) 2025-09-23 22:24:10 UTC
Nice work everyone!

Pushed to main for 25.11
Comment 14 Lucas Gass (lukeg) 2025-09-24 03:19:10 UTC
/kohadevbox/koha/xt/find-license-problems.t is failing now 

Is the right fix to update the test to allow for both 'http' and 'https' or find all the occurrences of: 

# <http://www.gnu.org/licenses>
Comment 15 Tomás Cohen Arazi (tcohen) 2025-09-24 14:01:24 UTC
(In reply to Lucas Gass (lukeg) from comment #14)
> /kohadevbox/koha/xt/find-license-problems.t is failing now 
> 
> Is the right fix to update the test to allow for both 'http' and 'https' or
> find all the occurrences of: 
> 
> # <http://www.gnu.org/licenses>

Following the spirit of this patches, we should enforce https for the GNU licenses. But I found a few places where my search didn't find some wrong cases. I'll attach a follow-up ASAP.
Comment 16 Tomás Cohen Arazi (tcohen) 2025-09-24 14:53:45 UTC
Created attachment 186890 [details] [review]
Bug 40163: (follow-up) Fix wrong GNU links and update test

Signed-off-by: Tomás Cohen Arazi <tomascohen@theke.io>
Comment 17 Lucas Gass (lukeg) 2025-09-24 15:26:53 UTC
follow-up pushed to main. Thanks Tomas!