-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Fixed an issue where checking domains raises an error with 'NoneType'… #21270
Fixed an issue where checking domains raises an error with 'NoneType'… #21270
Conversation
… object has no attribute 'split'
Thank you for your contribution. Your generosity and caring are unrivaled! Make sure to register your contribution by filling the Contribution Registration form, so our content wizard @YuvHayun will know he can start review the proposed changes. |
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.
Looks great! @spearmin10
Let's add one unit test to test cases where we try to extract a None type and make sure it won't raise an error.
For the Reviewer: Successfully created a pipeline in Gitlab with url: https://code.pan.run/xsoar/content/-/pipelines/3679914 |
@spearmin10 |
#21293) * Fixed an issue where checking domains raises an error with 'NoneType'… (#21270) * Fixed an issue where checking domains raises an error with 'NoneType' object has no attribute 'split' * Added a test case for domain * RN fixes Co-authored-by: Masahiko Inoue <54964121+spearmin10@users.noreply.github.com> Co-authored-by: YuvHayun <yhayun@paloaltonetworks.com>
… object has no attribute 'split'
Contributing to Cortex XSOAR Content
Make sure to register your contribution by filling the contribution registration form
The Pull Request will be reviewed only after the contribution registration form is filled.
Status
Related Issues
None
Description
When checking some domains (e.g.
mail16.amadeus.net
), it raises an error with'NoneType' object has no attribute 'split'
Screenshots
Minimum version of Cortex XSOAR
Does it break backward compatibility?
Must have