django-DefectDojo
django-DefectDojo copied to clipboard
[FIX] Reassign an engagement to another product
On the Engagement Edit page, it is possible to change the product. However, this has no effect on real engagement's product assignment. Adding the line to fix the issue.
https://github.com/DefectDojo/django-DefectDojo/issues/6588
I am not sure if this is a good idea. Deduplication works on a product level per default. So the old product might have findings that were marked as duplicates which are not duplicates anymore after moving the engagement and for the the new product deduplication would need to be run.
@X0x1RG9f Is there a strong use case to move an engagement to another product? If yes, the deduplication issue needs to be solved; if no we should remove the product
field from editing engagements.
This is a fix for #1512 that stopped working at some point
@Maffooch Well, yes, but aren't the problems with the deduplication relevant?
Most likely yes. I believe we could manually kick the dedupe action after saving the engagement, but I am not sure how effective it would be without modification.
I see Stefan's point, but I would assume most users would expect the copy to not modify data, rather than us trying to modify it smartly.
I know, this is already merged but I also suppose there should not be able easily to change the product for engagement.
There is high chance, that Engagement
contains Test
. This Test
may contain Finding
and Finding
may contain Endpoint
. But Endpoint
are directly connected to Product
, not only to Finding
. Change of Product
for Engagement
can create a lot of problems if nothing else is changed. Same for Product_API_Scan_Configuration
and DojoMeta
.