CaliDog / certstream-python

Python library for connecting to CertStream

MIT License

427 stars 73 forks source link

Removed signal import on win32 #67

Open rohanday3 opened 4 months ago

rohanday3 commented 4 months ago

Description

Issue: The current code does not run on Windows due to the absence of the signal module functionalities for setting SIGPIPE.

Solution: Added a conditional import and setting of signal only if the operating system is not Windows.

Changes

Added a check to determine the operating system.
Imported and set signal only for non-Windows systems.

Code


if sys.platform != 'win32':
    from signal import signal, SIGPIPE, SIG_DFL
    signal(SIGPIPE, SIG_DFL)

Testing

Verified that the code runs without errors on Windows. Ensured that the signal handling works correctly on non-Windows systems.

rohanday3 commented 2 months ago

Please merge