Add modules to unsafe globals

[mmaitre314]

• Modules added: commands, numpy.testing._private.utils, ssl, timeit
• Alphabetized modules

[Copilot]

Copilot reviewed 5 out of 6 changed files in this pull request and generated no comments.

Files not reviewed (1)

• setup.cfg: Language not supported

Comments suppressed due to low confidence (2)

tests/test_scanner.py:157

• There appears to be a duplicate or overwritten definition where class Malicious16 is first added (lines 116-130) and later replaced by class Malicious21. Consider removing the redundant Malicious16 to avoid confusion in the test payload definitions.

class Malicious21:

src/picklescan/scanner.py:144

• [nitpick] The modules in the unsafe globals mapping are intended to be alphabetized per the PR description; however, the current order does not strictly follow alphabetical order. Consider reordering the entries to enhance readability and maintain consistency.

"timeit": "*",

[Copilot]

Copilot reviewed 5 out of 6 changed files in this pull request and generated no comments.

Files not reviewed (1)

• setup.cfg: Language not supported

Comments suppressed due to low confidence (2)

tests/test_scanner.py:344

• [nitpick] Consider directly referencing ssl_mod.get_server_certificate instead of using getattr for improved clarity.

return (getattr(ssl_mod, "get_server_certificate"), ((fqdn, 443),))

src/picklescan/scanner.py:91

• [nitpick] Since the PR title indicates an alphabetized list of modules, please reorder the unsafe globals dictionary keys to maintain a consistent alphabetical order.

"numpy._core.multiarray": {