VAMDC / NodeSoftware

Python/Django-based software for running VAMDC data nodes.
http://www.vamdc.eu
GNU General Public License v3.0
15 stars 23 forks source link

Breakage in capabilities template #86

Closed guyrixon closed 12 years ago

guyrixon commented 12 years ago

The current Django template for the capabilities document has some problems with namespaces and schemata.

The template starts with this element:

<cap:capabilities xmlns:cap='http://www.ivoa.net/xml/VOSICapabilities/v1.0' xmlns:vs='http://www.ivoa.net/xml/VODataService/v1.0' xmlns:xsi='http://www.w3.org/2001/XMLSchema-instance' xmlns:tx='http://www.vamdc.org/xml/VAMDC-TAP/v1.0' xsi:schemaLocation='http://www.ivoa.net/xml/VOResource/v1.0 http://www.ivoa.net/xml/VOResource/v1.0 http://www.ivoa.net/xml/VODataService/v1.1 http://www.ivoa.net/xml/VODataService/VODataService-v1.0.xsd {{accessURL}}static/xsd/Capabilities.xsd'>

which has three problems:

  1. there's no entry in schemaLocation for http://www.vamdc.org/xml/VAMDC-TAP/v1.0;
  2. the syntax is wrong in the last line (single URL in place of URI-URL pair);
  3. Capabilities.xsd defines the wrong namespace - we need the other schema.

In fact, we now have enough schemata on web sites that we don't need to serve them from the nodes. This version of the first element works:

<cap:capabilities xmlns:cap='http://www.ivoa.net/xml/VOSICapabilities/v1.0' xmlns:vs='http://www.ivoa.net/xml/VODataService/v1.0' xmlns:xsi='http://www.w3.org/2001/XMLSchema-instance' xmlns:tx='http://www.vamdc.org/xml/VAMDC-TAP/v1.0' xsi:schemaLocation='http://www.ivoa.net/xml/VOResource/v1.0 http://www.ivoa.net/xml/VOResource/v1.0 http://www.ivoa.net/xml/VODataService/v1.1 http://www.ivoa.net/xml/VODataService/VODataService-v1.0.xsd http://www.vamdc.org/xml/VAMDC-TAP/v1.0 http://www.vamdc.org/xml/VAMDC-TAP/v1.0 http://www.ivoa.net/xml/VOSICapabilities/v1.0 http://vamdc.eu/downloads/xml/VOSI-capabilities-1.0.xsd'>

guyrixon commented 12 years ago

I've fixed this on my fork of the node software, and loaded the fixed template into the dev version of Chianti. The dev registry seems happy with the change.

ivh commented 12 years ago

merged this a while ago. closing issue.