Skip to content

Conversation

@SYNchroACK
Copy link

  • Rename properties types based on the naming pattern
  • Add missing _type field

@rpiazza
Copy link
Contributor

rpiazza commented Apr 9, 2025

@chisholm - is this worth merging?

@chisholm
Copy link
Contributor

In principle, I am okay with what the changes are doing. But this doesn't appear to change type names in enough places. For example, see imports in:

from .observables import (
URL, AlternateDataStream, ArchiveExt, Artifact, AutonomousSystem,
CustomObservable, Directory, DomainName, EmailAddress, EmailMessage,
EmailMIMEComponent, File, HTTPRequestExt, ICMPExt, IPv4Address,
IPv6Address, MACAddress, Mutex, NetworkTraffic, NTFSExt, PDFExt, Process,
RasterImageExt, SocketExt, Software, TCPExt, UNIXAccountExt, UserAccount,
WindowsPEBinaryExt, WindowsPEOptionalHeaderType, WindowsPESection,
WindowsProcessExt, WindowsRegistryKey, WindowsRegistryValueType,
WindowsServiceExt, X509Certificate, X509V3ExtensionsType,
)

This change breaks some of those imports. Perhaps an IDE should be used to do the refactor, to ensure the names are changed in all places they are used.

@CLAassistant
Copy link

CLAassistant commented Jul 31, 2025

CLA assistant check
All committers have signed the CLA.

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.

4 participants