-
Notifications
You must be signed in to change notification settings - Fork 5
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 Cloud Resource ID #46
base: main
Are you sure you want to change the base?
Conversation
thehappydinoa
commented
Oct 3, 2023
•
edited
edited
- Added Cloud Resource ID field to connectors
- Added Created At timestamp where applicable
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for this work! It should be pretty easy to merge with the in flight multiprocessing work. As this PR stands today, there are some significant merge conflicts with https://github.com/censys/censys-cloud-connector/blob/2df01cf29124ee1162e49c38c80b761eb0dab10b/src/censys/cloud_connectors/aws_connector/connector.py. I can figure out merging later next week when I get back to this work (currently working on bugs).
@ericbutera that sounds perfect, I was just about to push up some various test fixes. I think I will push the ones up for the base classes, but not the aws one particularly. Hopefully this works and if we run into any issues next week please let me know, I am happy to help resolve/rebase where needed. Thanks! Also I am going to keep it in draft form so it doesn't accidentally get merged. |