From 7cc6985ebd7bf9627d17bccdb72d5c132d756606 Mon Sep 17 00:00:00 2001
From: Jason Etheridge <jason@EquinoxOLI.org>
Date: Thu, 30 Jun 2022 21:25:50 -0400
Subject: [PATCH] SIP filters: fix whitespace here too

Signed-off-by: Jason Etheridge <jason@EquinoxOLI.org>
---
 SIPconfig.xml.example.evergreen | 298 ++++++++++++++++++++--------------------
 1 file changed, 148 insertions(+), 150 deletions(-)

diff --git a/SIPconfig.xml.example.evergreen b/SIPconfig.xml.example.evergreen
index 2cd196b..01f42ba 100644
--- a/SIPconfig.xml.example.evergreen
+++ b/SIPconfig.xml.example.evergreen
@@ -1,134 +1,134 @@
 <?xml version="1.0"?>
 <acsconfig xmlns="http://openncip.org/acs-config/1.0/">
 
-	<error-detect enabled="true" />
+  <error-detect enabled="true" />
 
   <!-- Set Net::Server::PreFork runtime parameters -->
-	<!--  <server-params
+  <!--  <server-params
            min_servers='1'
            min_spare_servers='0' /> -->
   
   
-	<listeners>
-		<service
-			port="0:8080/tcp"
-			transport="http"
-			protocol="NCIP/1.0" />
-
-		<service
-			port="8023/tcp"
-			transport="telnet"
-			protocol="SIP/1.00"
-			timeout="60" />
-
-		<service
-			port="127.0.0.1:6001/tcp"
-			transport="RAW" 
-			protocol="SIP/2.00"
-			timeout="60" />
-	</listeners>
-
-	<accounts>
-		<!--
-		Fine Item Detail returned by the Patron Information Request is
-		manufacturer-specific.  We support the following formats:
-		3m,  Swyer_A, Swyer_B, and EG_Legacy (default).
-		Specify which treatment you want in the av_format attribute.
-		For example: <login id="sc" password="pwd" institution="main" av_format="3m">
-		-->
-		<!--
-		The login attribute patron_status_always_permit_loans specifies whether
-		the charge privileges denied, renewal privilges denied, and
-		card reported lost flags in the patron status block should be
-		coerced to permissive values regardless of the actual state
-		of the patron record. Turning this on works around an issue
-		where a 2019-12 change by the Hoopla SIP2 client takes those flag
-		fields into account, but some libraries may not wish those
-		to block a patron's access to online resources that use
-		SIP2 to authenticate. This setting can also be set as
-		an implementation_config option; note that if it is set to
-		'true' or 'false' as a login attribute, the login attribute will
-		override whatever is set in the implementation_config.
-		-->
-		<login id="scclient" password="clientpwd" institution="gapines"/>
-	</accounts>
-
-	<!-- Institution tags will hold stuff used to interface to -->
-	<!-- the rest of the ILS: authentication parameters, etc.  I -->
-	<!-- don't know what yet, so it'll just be blank.  But there -->
-	<!-- needs to be one institution stanza for each institution -->
-	<!-- named in the accounts above. -->
-	<institutions>
-
-
-		<institution id="gapines" implementation="OpenILS::SIP">
-
-			<!-- This defines what actions we want to allow 
-				remote clients (self-check machines) to perform -->
-			<policy 
-				checkin="true" 
-				checkout="true" 
-				renewal="true" 
-				status_update="false" 
-				offline="false" 
-				timeout="600" 
-				retries="3"/>
-	
-			<!-- The default encoding defined in the SIP specification is -->
-			<!-- ASCII, which isn't great for French, Spanish, Armenian. -->
-			<!-- You can specify a different encoding here, based on the -->
-			<!-- encodings supported by your SIP client and your Encode -->
-			<!-- module; run the following command to get a list of supported -->
-			<!-- encodings: -->
-			<!--   perl -MEncode -le "print for Encode->encodings(':all')" -->
-
-			<!-- UTF-8 is the recommended encoding if your SIP client supports it -->
-			<encoding>ascii</encoding>
-
-			<!--
-				When set to true, return the hold notification phone number in the patron home phone (BF) field for each Checkin Response (10).
-				This is an unsupported extention to SIP2 and potentially exposes additional user infomation, do not enable it unless required locally.
-			-->
-			<!--
-			<phone_ext_to_msg10 enabled="false" />
-			-->
-
-			<!-- implementation specific config options go here -->
-			<implementation_config>
-				<bootstrap>SYSCONFDIR/opensrf_core.xml</bootstrap>
-				<currency>USD</currency>
-
-				<!-- These defines what this SIP code has the ability to support -->
-				<supports>
-					<item name='magnetic media' value='true'/>
-					<item name='security inhibit' value='false'/>
-					<item name='offline operation' value='false'/>
-					<item name='patron status request' value='true'/>
-					<item name='checkout' value='true'/>
-					<item name='checkin' value='true'/>
-					<item name='block patron' value='true'/>
-					<item name='acs status' value='true'/>
-					<item name='login' value='true'/>
-					<item name='patron information' value='true'/>
-					<item name='end patron session' value='true'/>
-					<item name='fee paid' value='true'/>
-					<item name='item information' value='true'/>
-					<item name='item status update' value='false'/>
-					<item name='patron enable' value='false'/>
-					<item name='hold' value='false'/>
-					<item name='renew' value='true'/>
-					<item name='renew all' value='true'/>
-				</supports>
-				<options>
-					<!-- msg64, the patron information request can be
-					  made to return item barcodes by setting
-					  the option 'msg64_summary_datatype' to 'barcode'
-					  as below. Any other value, or no value at all
-					  will cause OpenILS::SIP to return the title
-					  in response to a message 64 request, which was the
-					  default behaviour in previous versions of Evergreen.
-					-->
-					<option name='msg64_summary_datatype' value='barcode' />
+  <listeners>
+    <service
+      port="0:8080/tcp"
+      transport="http"
+      protocol="NCIP/1.0" />
+
+    <service
+      port="8023/tcp"
+      transport="telnet"
+      protocol="SIP/1.00"
+      timeout="60" />
+
+    <service
+      port="127.0.0.1:6001/tcp"
+      transport="RAW" 
+      protocol="SIP/2.00"
+      timeout="60" />
+  </listeners>
+
+  <accounts>
+    <!--
+    Fine Item Detail returned by the Patron Information Request is
+    manufacturer-specific.  We support the following formats:
+    3m,  Swyer_A, Swyer_B, and EG_Legacy (default).
+    Specify which treatment you want in the av_format attribute.
+    For example: <login id="sc" password="pwd" institution="main" av_format="3m">
+    -->
+    <!--
+    The login attribute patron_status_always_permit_loans specifies whether
+    the charge privileges denied, renewal privilges denied, and
+    card reported lost flags in the patron status block should be
+    coerced to permissive values regardless of the actual state
+    of the patron record. Turning this on works around an issue
+    where a 2019-12 change by the Hoopla SIP2 client takes those flag
+    fields into account, but some libraries may not wish those
+    to block a patron's access to online resources that use
+    SIP2 to authenticate. This setting can also be set as
+    an implementation_config option; note that if it is set to
+    'true' or 'false' as a login attribute, the login attribute will
+    override whatever is set in the implementation_config.
+    -->
+    <login id="scclient" password="clientpwd" institution="gapines"/>
+  </accounts>
+
+  <!-- Institution tags will hold stuff used to interface to -->
+  <!-- the rest of the ILS: authentication parameters, etc.  I -->
+  <!-- don't know what yet, so it'll just be blank.  But there -->
+  <!-- needs to be one institution stanza for each institution -->
+  <!-- named in the accounts above. -->
+  <institutions>
+
+
+    <institution id="gapines" implementation="OpenILS::SIP">
+
+      <!-- This defines what actions we want to allow 
+        remote clients (self-check machines) to perform -->
+      <policy 
+        checkin="true" 
+        checkout="true" 
+        renewal="true" 
+        status_update="false" 
+        offline="false" 
+        timeout="600" 
+        retries="3"/>
+  
+      <!-- The default encoding defined in the SIP specification is -->
+      <!-- ASCII, which isn't great for French, Spanish, Armenian. -->
+      <!-- You can specify a different encoding here, based on the -->
+      <!-- encodings supported by your SIP client and your Encode -->
+      <!-- module; run the following command to get a list of supported -->
+      <!-- encodings: -->
+      <!--   perl -MEncode -le "print for Encode->encodings(':all')" -->
+
+      <!-- UTF-8 is the recommended encoding if your SIP client supports it -->
+      <encoding>ascii</encoding>
+
+      <!--
+        When set to true, return the hold notification phone number in the patron home phone (BF) field for each Checkin Response (10).
+        This is an unsupported extention to SIP2 and potentially exposes additional user infomation, do not enable it unless required locally.
+      -->
+      <!--
+      <phone_ext_to_msg10 enabled="false" />
+      -->
+
+      <!-- implementation specific config options go here -->
+      <implementation_config>
+        <bootstrap>SYSCONFDIR/opensrf_core.xml</bootstrap>
+        <currency>USD</currency>
+
+        <!-- These defines what this SIP code has the ability to support -->
+        <supports>
+          <item name='magnetic media' value='true'/>
+          <item name='security inhibit' value='false'/>
+          <item name='offline operation' value='false'/>
+          <item name='patron status request' value='true'/>
+          <item name='checkout' value='true'/>
+          <item name='checkin' value='true'/>
+          <item name='block patron' value='true'/>
+          <item name='acs status' value='true'/>
+          <item name='login' value='true'/>
+          <item name='patron information' value='true'/>
+          <item name='end patron session' value='true'/>
+          <item name='fee paid' value='true'/>
+          <item name='item information' value='true'/>
+          <item name='item status update' value='false'/>
+          <item name='patron enable' value='false'/>
+          <item name='hold' value='false'/>
+          <item name='renew' value='true'/>
+          <item name='renew all' value='true'/>
+        </supports>
+        <options>
+          <!-- msg64, the patron information request can be
+            made to return item barcodes by setting
+            the option 'msg64_summary_datatype' to 'barcode'
+            as below. Any other value, or no value at all
+            will cause OpenILS::SIP to return the title
+            in response to a message 64 request, which was the
+            default behaviour in previous versions of Evergreen.
+          -->
+          <option name='msg64_summary_datatype' value='barcode' />
 
 
                     <!--
@@ -147,10 +147,10 @@
                     <option name='msg64_hold_items_available' value='true' />
                     -->
 
-					<!--
-						If enabled, the PC field in patron-info requests will return the non-translated profile name
-					<option name='patron_type_uses_code' value='true' />
-					-->
+          <!--
+            If enabled, the PC field in patron-info requests will return the non-translated profile name
+          <option name='patron_type_uses_code' value='true' />
+          -->
 
                     <!--
                         By default, most dates use the SIP date format.  Some,
@@ -160,26 +160,26 @@
                     <option name='use_sip_date_format' value='true' />
                     -->
 
-					<!--
-						If enabled, return the calculated value for the recall
-						flag instead of always returning not-OK
-					<option name='patron_calculate_recal_ok' value='true' />
-					-->
+          <!--
+            If enabled, return the calculated value for the recall
+            flag instead of always returning not-OK
+          <option name='patron_calculate_recal_ok' value='true' />
+          -->
 
-					<!-- see description of patron_status_always_permit_loans in the login section -->
-					<!--
-					<option name='patron_status_always_permit_loans' value='false' />
-					-->
+          <!-- see description of patron_status_always_permit_loans in the login section -->
+          <!--
+          <option name='patron_status_always_permit_loans' value='false' />
+          -->
 
-					<!--
-						Allow patrons to connect to SIP services using their
-						username in addition to their barcode.  See the
-						org unit setting 'opac.barcode_regex' for configuring
-						what constitutes a barcode vs a username.
-					<option name='support_patron_username_login' value='false' />
-					-->
+          <!--
+            Allow patrons to connect to SIP services using their
+            username in addition to their barcode.  See the
+            org unit setting 'opac.barcode_regex' for configuring
+            what constitutes a barcode vs a username.
+          <option name='support_patron_username_login' value='false' />
+          -->
 
-				</options>
+        </options>
 
                 <checkin_override>
                     <event>COPY_ALERT_MESSAGE</event>
@@ -199,11 +199,9 @@
                 <checkin_hold_as_transit>1</checkin_hold_as_transit>
                 -->
 
-			</implementation_config>
-	
-		</institution>
-	
-	</institutions>
+      </implementation_config>
+  
+    </institution>
+  
+  </institutions>
 </acsconfig>
-
-<!-- vim:noet:ts=4:sw=4: -->
-- 
2.11.0