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

WD 05 SCO #285

Merged
merged 19 commits into from
Sep 18, 2019
Merged

WD 05 SCO #285

merged 19 commits into from
Sep 18, 2019

Conversation

khdesai
Copy link
Contributor

@khdesai khdesai commented Aug 19, 2019

No description provided.

@rpiazza
Copy link
Contributor

rpiazza commented Aug 19, 2019

Is there a reason why the process observable's ID is specified as:

    ('id', IDProperty(_type)),

instead of

    ('id', IDProperty(_type, spec_version='2.1')),

@khdesai
Copy link
Contributor Author

khdesai commented Aug 19, 2019

Is there a reason why the process observable's ID is specified as:

    ('id', IDProperty(_type)),

instead of

    ('id', IDProperty(_type, spec_version='2.1')),

No reason, that's just a relic of an old way I used to separate the Process SCO from the other SCOs when it came time to creating a deterministic ID (since Process doesn't use deterministic IDs). I've added the spec_version back in. Thanks for catching that.

@emmanvg emmanvg added this to the 1.2.0 milestone Aug 21, 2019
stix2/base.py Outdated Show resolved Hide resolved
stix2/base.py Outdated Show resolved Hide resolved
stix2/base.py Outdated Show resolved Hide resolved
stix2/base.py Outdated Show resolved Hide resolved
stix2/base.py Outdated Show resolved Hide resolved
stix2/base.py Outdated Show resolved Hide resolved
stix2/base.py Outdated Show resolved Hide resolved
stix2/base.py Outdated Show resolved Hide resolved
Copy link
Contributor

@clenk clenk left a comment

Choose a reason for hiding this comment

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

There are some common properties we need to add to all SCOs: spec_version, object_marking_refs, granular_markings, defanged.

EmailMessage is missing the message_id property.

Also, in 2.1 *_ref[s] properties now contain identifiers rather than object-refs. So for example instead of "resolves_to_refs": ["1"] it would be "resolves_to_refs": ["ipv4-addr--<uuid>"].

stix2/v21/observables.py Outdated Show resolved Hide resolved
stix2/v21/observables.py Outdated Show resolved Hide resolved
stix2/v21/observables.py Outdated Show resolved Hide resolved
stix2/base.py Show resolved Hide resolved
@codecov-io
Copy link

codecov-io commented Sep 11, 2019

Codecov Report

Merging #285 into stix2.1 will decrease coverage by 0.91%.
The diff coverage is 76.79%.

Impacted file tree graph

@@             Coverage Diff             @@
##           stix2.1     #285      +/-   ##
===========================================
- Coverage    99.01%   98.09%   -0.92%     
===========================================
  Files          121      123       +2     
  Lines        12924    13409     +485     
===========================================
+ Hits         12797    13154     +357     
- Misses         127      255     +128
Impacted Files Coverage Δ
stix2/test/v21/test_datastore_filters.py 100% <ø> (ø) ⬆️
stix2/v21/sdo.py 95.29% <ø> (ø) ⬆️
stix2/test/v21/test_utils.py 100% <ø> (ø) ⬆️
stix2/v21/common.py 100% <ø> (ø) ⬆️
stix2/v20/common.py 100% <ø> (ø) ⬆️
stix2/test/v21/test_core.py 100% <ø> (ø) ⬆️
stix2/v20/sdo.py 100% <ø> (ø) ⬆️
stix2/test/v20/test_utils.py 100% <ø> (ø) ⬆️
stix2/v21/observables.py 96.96% <100%> (+0.6%) ⬆️
stix2/test/v21/test_observed_data.py 100% <100%> (ø) ⬆️
... and 11 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d2a649b...3b1c922. Read the comment docs.

Copy link
Contributor

@clenk clenk left a comment

Choose a reason for hiding this comment

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

Thanks @khdesai!

@clenk clenk merged commit e1a88a4 into oasis-open:stix2.1 Sep 18, 2019
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.

6 participants