search

plantbreeding / IPK-BrAPI-Validator

MIT License
4 stars 3 forks source link

Dynamically detect and test the correct BrAPI version supported by the server #42

Open ClayBirkett opened 5 years ago

ClayBirkett commented 5 years ago

The BrAPI Validator at http://webapps.ipk-gatersleben.de/brapivalidator/ does its automated weekly test using v1.2. Can this be changed to v1.3? Better yet, can it read the version from the calls endpoint and test at the highest version supported?

langeipk commented 4 years ago

Indeed, the BrAPI Validator does not set the BrAPI version for the scheduled test dynamical from the results of the calls call. Rather the the property "testedVersion" in config.properties is used to set one version for all endpoints.

Furthermore, not all endpoints release version information in the calls call. Such as: http://www.cropontology.org/brapi/v1/calls

Nevertheless, this feature would be a nice add-on to support multiple versions for each endpoint or alternatively the most recent one, as mentioned by @ClayBirkett.

So pull requests are welcome.