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

mustUseAdvisory header logic #26

Open
bencart opened this issue Jul 24, 2017 · 0 comments
Open

mustUseAdvisory header logic #26

bencart opened this issue Jul 24, 2017 · 0 comments

Comments

@bencart
Copy link
Contributor

bencart commented Jul 24, 2017

There is an issue with the way the mustUseAdvisory header is being handled in a provider.

Current Logic:

If object has an advisory ref id
    if mustUseAdvisory = true
       process request
    else
       raise error
else
    if mustUseAdvisory = true
       raise error
    else
       process request

Expected Behaviour:

if mustUseAdvisory = true
   if there is a refId clash or the object does not have an advisory ref id
      raise error
   else
      process request
else
   process request assigning a new refid

The response to a create many where the mustUseAdvisory header is unset or set to false should contain a list of "Create Responses" containing advisory refId and it's new refid.

joerghuber added a commit that referenced this issue Mar 25, 2019
Add sample Consumers to the HITS Consumer demo project
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

No branches or pull requests

1 participant