Bug 18245

Summary: debian/templates/koha-conf-site.xml.in missing tls
Product: Koha Reporter: Mark Tompsett <mtompset>
Component: Architecture, internals, and plumbingAssignee: Bugs List <koha-bugs>
Status: RESOLVED DUPLICATE QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: mirko
Version: master   
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15427
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 18245: Add missing TLS to debian/templates/koha-conf-site.xml.in

Description Mark Tompsett 2017-03-10 19:56:07 UTC
As noted by Marcel in https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15427#c17
Comment 1 Mark Tompsett 2017-03-10 19:59:45 UTC
Created attachment 61004 [details] [review]
Bug 18245: Add missing TLS to debian/templates/koha-conf-site.xml.in

This was found as a result of testing another bug.
Added missing line.
Comment 2 Mirko Tietgen 2017-03-22 16:23:39 UTC
If you want to add this, I think you also need to add <ca>, <cert> and <key>?
Comment 3 Mark Tompsett 2017-04-14 16:04:06 UTC
(In reply to Mirko Tietgen from comment #2)
> If you want to add this, I think you also need to add <ca>, <cert> and <key>?

You may be right, but check out the diff between etc/koha-conf.xml and debian/templates/koha-conf-site.xml/in currently.

I'm not sure the correct way to add, because hard-coded-path/__KOHASITE__/rest-of-path is not the same as __SOME_THING_TO_FULLY_REPLACE__/rest-of-path.

Thoughts?
Comment 4 Katrin Fischer 2020-04-19 11:04:58 UTC
Checked the file in question and found:

 <tls>__DB_USE_TLS__</tls>
 <ca>__DB_TLS_CA_CERTIFICATE__</ca>
 <cert>__DB_TLS_CLIENT_CERTIFICATE__</cert>
 <key>__DB_TLS_CLIENT_KEY__</key>

*** This bug has been marked as a duplicate of bug 22857 ***