Skip to content
This repository has been archived by the owner on May 29, 2024. It is now read-only.

Unpin stix2 and fix breaking changes #162

Merged
merged 2 commits into from
Aug 24, 2021
Merged

Conversation

lava
Copy link
Member

@lava lava commented Aug 23, 2021

📔 Description

The StixBase::object_properties() method was removed in stix 3.0 on charges of being obsolete, breaking our code.

This PR removes the calls to object_properties() analogous to how it was done in the linked upstream PR, and bumps the minimum dependency version to 3.0 since making these changes will break compatibility with stix 2.x,

📝 Checklist

  • All user-facing changes have changelog entries.
  • The changes are reflected on docs.tenzir.com/threatbus, if necessary.
  • The PR description contains instructions for the reviewer, if necessary.

🎯 Review Instructions

@lava lava force-pushed the story/ch27394/stix3-update branch 2 times, most recently from b1494db to e6b9cc1 Compare August 23, 2021 18:31
Benno Evers added 2 commits August 23, 2021 20:33
This function was removed upstream from STIX 3.0
in 2cda97cf5e51403c43a96ceb9073afab7706638a, since
it apparently became obsolete.
@lava lava force-pushed the story/ch27394/stix3-update branch from e6b9cc1 to 59ebf76 Compare August 23, 2021 18:34
Copy link

@rolandpeelen rolandpeelen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change lgtm. I've grepped through the codebase and didn't find any leftover object_properties, and since the test pass, I'll approve. Have not ran it locally though :)

@lava lava merged commit 053703b into master Aug 24, 2021
@lava lava deleted the story/ch27394/stix3-update branch August 24, 2021 11:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants