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.

Sign up for GitHub

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

Jump to bottom

No error when merging two entities with a higher confidence level #6502

Closed
yassine-ouaamou opened this issue Mar 27, 2024 · 0 comments
Closed

No error when merging two entities with a higher confidence level #6502

yassine-ouaamou opened this issue Mar 27, 2024 · 0 comments
Assignees
@SarahBocognano
Labels
bug use for describing something not working as expected solved use to identify issue that has been solved (must be linked to the solving PR)
Milestone
Release 6.0.10

Comments

@yassine-ouaamou
Copy link
Member

Description

When merging two entities with a confidence level higher than the users, no error is triggered and the task seems complete

Environment

Testing (6.0.8)

Reproducible Steps

Steps to create the smallest reproducible scenario:

  1. Make sure to have a user having confidence level = 50
  2. Go to Data > Entities, filter on intrusion sets with a confidence level = 100
  3. Select two intrusion sets and Merge them

Expected Output

Be informed that the merge is not possible

Actual Output

The merge seems complete and no error is triggered
image

Additional information

Screenshots (optional)
@yassine-ouaamou yassine-ouaamou added bug use for describing something not working as expected needs triage use to identify issue needing triage from Filigran Product team labels Mar 27, 2024
@SamuelHassine SamuelHassine added this to the Release 6.0.9 milestone Mar 28, 2024
@jborozco jborozco removed the needs triage use to identify issue needing triage from Filigran Product team label Mar 28, 2024
@jborozco jborozco mentioned this issue Mar 28, 2024
Closed
@SarahBocognano SarahBocognano self-assigned this Mar 28, 2024
@SamuelHassine SamuelHassine modified the milestones: Release 6.0.9, Release 6.0.10 Apr 3, 2024
@SarahBocognano SarahBocognano mentioned this issue Apr 4, 2024
Merged
SarahBocognano added a commit that referenced this issue Apr 5, 2024
@SarahBocognano
[backend] Fix No error when merging two entities with a higher confid…
a6741ed 
…ence level (#6502)
@SamuelHassine SamuelHassine added the solved use to identify issue that has been solved (must be linked to the solving PR) label Apr 5, 2024
@SamuelHassine SamuelHassine modified the milestones: Release 6.0.10, Release 6.1.0 Apr 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@SarahBocognano SarahBocognano

Labels
bug use for describing something not working as expected solved use to identify issue that has been solved (must be linked to the solving PR)
Projects
None yet
Milestone
Release 6.0.10
Development

No branches or pull requests
4 participants
@SamuelHassine @yassine-ouaamou @jborozco @SarahBocognano