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.

Sign up for GitHub

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

Jump to bottom

Added missing attributes to the Case class. #50

Merged
merged 1 commit into from
Dec 4, 2017
Merged

Added missing attributes to the Case class. #50

merged 1 commit into from
Dec 4, 2017

Conversation

npratley
Copy link

@npratley npratley commented Dec 4, 2017

The Case class's constructor did not add all attributes present in the JSON data returned from the API. For example, the id and caseId. You can now do:

case = thehive.case('some_case_id')
print(case.id)
print(case.caseId)  # The ID as shown in the web UI
print(case.status)
print(case.owner)

@npratley
Added missing attributes to the Case class. These attributes are pres…
b44541d 
…ent in the JSON returned from the API but were not set in the constructor.
@nadouani nadouani merged commit 0ec8171 into TheHive-Project:develop Dec 4, 2017
nadouani added a commit that referenced this pull request Dec 4, 2017
@nadouani
#50 Remove user case attribute
ee864db
@nadouani nadouani added this to the 1.4.0 milestone Dec 4, 2017
@npratley npratley mentioned this pull request Dec 4, 2017
Closed
@npratley npratley deleted the missing-attributes branch December 4, 2017 23:40
@npratley npratley restored the missing-attributes branch December 4, 2017 23:40
nadouani added a commit that referenced this pull request Dec 5, 2017
@nadouani
#50 Revert back the changes that added new case attributes
aaef126
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
1.4.0
Development

Successfully merging this pull request may close these issues.
2 participants
@npratley @nadouani