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

Add adapter modules and methods for mixbox modules/methods that break backwards compatibility #280

Open
bworrell opened this issue Oct 29, 2015 · 3 comments

Comments

@bworrell
Copy link
Contributor

There are a few things we've been cooking up in mixbox that may be incompatible with existing, documented code.

An example is the idgen module [1], which was moved to mixbox, but now accepts different arguments (was a dictionary, now a mixbox.namespaces.Namespace object).

We should try to write adapters where possible and limit the impact of moving to mixbox as much as we can.

[1] http://stix.readthedocs.org/en/stable/overview/id_namespaces.html#id-namespaces

@abulhol
Copy link

abulhol commented Nov 3, 2015

I appreciate your work on this. At the moment, I am pretty confused by idgen appearing in maec.utils, mixbox and cybox.utils and not working as expected. I get errors on set_id_namespace: "Must be a Namespace object", and set_id_method(IDGenerator.METHOD_INT) results in UUID instead of integer IDs (except if I use the Master branch).

@bworrell
Copy link
Contributor Author

bworrell commented Nov 3, 2015

@abulhol, If you are interested in using python-stix/python-cybox/python-maec, we recommend heading on over to PyPI and downloading an official release (or using pip install maec) rather than trying to use the latest commits from the repository.

If you are running code off of a branch on GitHub (including master) you are working from in-development code (with a x.y.z.dev[number] version number) which may not work the way you want it to :)

>>> import cybox
>>> print cybox.__version__
2.1.0.13.dev0

Take a look at the python-maec installation documentation for more details.

@abulhol
Copy link

abulhol commented Nov 4, 2015

Sorry, I messed up the imports. Now it works.

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

2 participants