nasa-pds-engineering-node / pds-registry-app

(deprecated) See https://github.com/NASA-PDS/registry for new and improved capability.
https://nasa-pds.github.io/registry/
Other
1 stars 1 forks source link

harvest flags as mandatory a config elements the docs describe as optional #220

Closed msbentley closed 2 years ago

msbentley commented 2 years ago

🐛 Describe the bug

Using harvest 3.5.1 if I do not include the <registry> config element I get an error:

[ERROR] Missing required '/harvest/registry' element

however, the docs state that this element is optional.

🕵️ Expected behavior

Doc and executable should be aligned.

📚 Version of Software Used

harvest 3.5.1

🖥 System Info


🦄 Related requirements

⚙️ Engineering Details

jordanpadams commented 2 years ago

@tdddblog I seem to remember authorization being optional, but that may have changed. depending on what is actually required or not, let's update.

tdddblog commented 2 years ago

@jordanpadams It is not authorization. It is registry (Elasticsearch) configuration. It is now required to handle LDDs (XSDs) in schemaLocation attribute.

msbentley commented 2 years ago

So, to confirm, is a running and accessible registry required to run harvest?

tdddblog commented 2 years ago

Yes. I will update documentation.

nutjob4life commented 2 years ago

Documentation updates in progress.

nutjob4life commented 2 years ago

Documentation updates in progress.