Ticket #458 (closed defect: fixed)

Opened 1 year ago

Last modified 5 months ago

Add comments to publishing default.xml to clarify what the elements are for.

Reported by: amit Assigned to: amit
Priority: critical Milestone:
Component: ambra Version: 0.7
Keywords: configuration defaults publishing Cc:
Blocking: Blocked By:

Description

org.plos.configuration.defaults.xml have a bunch of defaults defined with no comments indicating what they are for. We want to convey what the information there is being used for rather than just how to modify it for a particular configuration. This way at least end-users will have a slightly better idea on how to change things for their environment.

Dependency Graph

Change History

08/06/07 22:15:36 changed by stevec

In [3292], added comments to all the ones that I can reliably comment on. Things have changed enough through various re-workings that I do not know the purpose and usage of all the elements anymore.

08/06/07 22:16:04 changed by stevec

  • owner changed from stevec to amit.

08/06/07 22:22:00 changed by amit

  • owner changed from amit to stevec.

Even these:

<config>
 	  <cas>
 	    <url>
 	      <base>https://${network.hosts.cas}:${network.ports.cas-ssl}/cas</base>
 	      <login>${cas.url.base}/login</login>
 	      <logout>${cas.url.base}/logout</logout>
 	      <proxy-receptor>${cas.url.base}/proxy</proxy-receptor>
 	      <proxy-validate>${cas.url.base}/proxyValidate</proxy-validate>
 	    </url>
 	  </cas>
 	
 	  <pub>
 	    <host>${network.hosts.default}:${network.ports.default}</host>
 	    <port>
 	      <http>${network.ports.default}</http>
 	      <https>${network.ports.plosone-ssl}</https>
 	    </port>
 	
 	    <url><proxy-callback>https://${network.hosts.plosone}:${network.ports.plosone-ssl}${pub.app-context}/CasProxyServlet</proxy-callback></url>

38

08/06/07 22:23:17 changed by stevec

yes. even those. Pradeep know those better than me. And the whole CAS thing should probably just remove proxying support since we don't need it anymore.

08/06/07 22:25:07 changed by amit

  • owner changed from stevec to pradeep.

08/10/07 18:50:49 changed by pradeep

(In [3411]) Correct the pdpName config entry for user service in the global-defaults. Remove the pdpName config overrides from other configs including the one in the packages.

Also removed the implied permissions for Fedora operations since the access service is no longer in use with Fedora.

Addresses #458

08/10/07 18:51:32 changed by pradeep

  • owner changed from pradeep to amit.

08/10/07 18:59:20 changed by amit

  • owner changed from amit to pradeep.

I think the following still need comments on what exactly is the use:

<cas>
 	    <url>
 	      <base>https://${network.hosts.cas}:${network.ports.cas-ssl}/cas</base>
 	      <login>${cas.url.base}/login</login>
 	      <logout>${cas.url.base}/logout</logout>
 	      <proxy-receptor>${cas.url.base}/proxy</proxy-receptor>
	      <proxy-validate>${cas.url.base}/proxyValidate</proxy-validate>
	    </url>
</cas>

According to Steve you know these better than he does.

08/11/07 00:52:03 changed by pradeep

(In [3413]) Additional docs for CAS config - Addresses #458

08/11/07 00:52:45 changed by pradeep

  • owner changed from pradeep to amit.

08/11/07 08:35:30 changed by amit

  • status changed from new to closed.
  • resolution set to fixed.

Thanks.

07/16/08 11:00:34 changed by

  • milestone deleted.

Milestone 0.8 deleted