Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

VirES WPS fixes #121

Merged
merged 6 commits into from
Feb 20, 2025
Merged

VirES WPS fixes #121

merged 6 commits into from
Feb 20, 2025

Conversation

pacesm
Copy link
Collaborator

@pacesm pacesm commented Feb 20, 2025

This PR introduces a couple of fixes to the WPS client:

  • POST requests are now sent with the proper application/xml content type header. The server may start enforcing request with the proper content type in the future.
  • The response references may contain href attribute with the namespace prefix xlink:href. The used WPS 1.0 standard seem to be somewhat ambiguous which one is correct and therefore, to make the parser more robust, both variants are accepted.
  • Retry requests for the failed status update have been implemented. The client now repeats the failed job status polling 3 times with 20 seconds interval.
  • In order to be able to inform users about the failed requests, the default logging level has been changed from NO_LOGGING to ERROR.
  • The client now handles response references as relative URLs (encountered in local development environment).

@smithara smithara merged commit af39718 into staging Feb 20, 2025
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants