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

Expand Status Resource to include message for pending and successes statuses #65

Closed
marlontaylor opened this issue Apr 18, 2018 · 3 comments
Closed

Comments

@marlontaylor
Copy link

@marlontaylor marlontaylor commented Apr 18, 2018

The Status Resource provides detailed information for failures via a list of status-failure type which contains:

  • id - The identifier of the object that failed to be created. For STIX objects the id MUST be the STIX Object id. For object types that do not have their own identifier, the server MAY use any value as the id.
  • message - A message indicating why the object failed to be created.

While the successes and pending statuses only contain a list of stings:

  • string - For STIX objects the STIX ID MUST be used here. For object types that do not have their own identifier, the server MAY use any value as the id.

successes and pending should include the ability to provide a message.

Options:

Backward Compatible - expand the Status Response to include two optional keys (pending_responses and success_responses) which will contain a list of status-failure to provide detailed information for successes and pending statuses.

  • the next major version of TAXII will implement this change and depreciate the list of strings

Non-Backward Compatible - update the current successes and pending to use status-failure.

@gtback
Copy link

@gtback gtback commented Apr 23, 2018

For clarity, if we do use the non-backward-compatible option, we might want to rename the status-failure structure to something that's not specific to failures.

@jordan2175
Copy link

@jordan2175 jordan2175 commented May 15, 2018

We discussed this on the working call on 2018-05-15 and the consensus was to accept this change, however, there was not a decision on how this should be done. The editors will work with Marlon to come up with a final proposal for the TC.

@jordan2175
Copy link

@jordan2175 jordan2175 commented May 17, 2018

I have made changes in section 4.3 to address this.

@jordan2175 jordan2175 added this to In progress in TAXII-2.1 May 17, 2018
@jordan2175 jordan2175 closed this May 18, 2018
@jordan2175 jordan2175 moved this from In progress to Done in TAXII-2.1 May 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
TAXII-2.1
  
Done
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants