Skip to content
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

Fix bug where integration returns no result if the lookup value is not the indicator's primary value #8

Merged
merged 5 commits into from
Feb 1, 2024

Conversation

ajung44
Copy link
Contributor

@ajung44 ajung44 commented Jan 30, 2024

An Indicator might have multiple file hash values associated with it such as SHA256, MD5, SHA1, etc. However, the indicator's primary value can only be one of them.

If the indicator's primary value is SHA256, and user searched with its MD5 value, the integration returns no result as it fails to match the searched MD5 value. Applied fix to iterate through the associated values to match with the search value instead.

penwoodjon and others added 5 commits January 23, 2023 11:12
Updating Github Actions & Adding config.json
… Machine Readability

Creating Release with Dereferenced Symlinks, `polarityIntegrationUuid`, & No Dev Dependencies for Machine Readability
Removing rejectUnauthorized from config files
INT-1129: Fix onDetails lookup for non CVEs
…t the indicator's primary value, and instead alternative value listed
@sarus
Copy link
Contributor

sarus commented Feb 1, 2024

@ajung44 Thank you so much for the PR! Can you retarget the PR onto the develop branch? Once we merge into develop we'll create a new release and tag it.

@ajung44 ajung44 changed the base branch from master to develop February 1, 2024 20:30
@ajung44
Copy link
Contributor Author

ajung44 commented Feb 1, 2024

@ajung44 Thank you so much for the PR! Can you retarget the PR onto the develop branch? Once we merge into develop we'll create a new release and tag it.

Yep, changed the target to develop branch!

@sarus sarus merged commit 9223d9f into polarityio:develop Feb 1, 2024
1 check passed
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