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

handle errors returned by the geonetwork POST /rest/records entrypoint #2

Merged
merged 4 commits into from
Jan 31, 2025

Conversation

mki-c2c
Copy link
Member

@mki-c2c mki-c2c commented Jan 31, 2025

Implementation of logics to recognize error condition in the response from geonetwork API post record call.

All details of the error detection are handled in this lib, the parent app will only handle the ParamException

@mki-c2c mki-c2c changed the title handel errors returned by the geonetwork POST /rest/records entrypoint handle errors returned by the geonetwork POST /rest/records entrypoint Jan 31, 2025
@mki-c2c mki-c2c requested a review from jeanmi151 January 31, 2025 10:58
@jeanmi151
Copy link
Contributor

would be great to have that for all function with pasting the code

@mki-c2c
Copy link
Member Author

mki-c2c commented Jan 31, 2025

Yes, you're right.
Do all GN API calls return HTTP 200 and this type of error structure ?
is it documented in GN source code, to implement it the correct way ?

@jeanmi151
Copy link
Contributor

Yes, you're right. Do all GN API calls return HTTP 200 and this type of error structure ? is it documented in GN source code, to implement it the correct way ?

Not sure maybe some clue in here https://georchestra-127-0-0-1.nip.io/geonetwork/doc/api/index.html

@mki-c2c
Copy link
Member Author

mki-c2c commented Jan 31, 2025

thanks
in the schema , "stack" does not seem to be specified, so I'll take it as optional

@mki-c2c mki-c2c merged commit 5175667 into main Jan 31, 2025
1 check passed
@mki-c2c mki-c2c deleted the gn_record_post_error_handling branch January 31, 2025 15:41
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