Skip to content

Remove cross-DB transactions against dast_site_profiles

Arpit Gogia requested to merge 480014-fix-cross-db-dast-site-profiles into master

What does this MR do and why?

Resolves the cross DB transaction happening when a DAST site profile is created or updated.

As DAST related tables are moved to the gitlab_sec schema, the point of contention was the audit log getting created within the same transaction as the site profile is getting created/updated. That query has now been moved outside. Moreover, the audit event will now only be created if the create/update transaction is successful.

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #480014 (closed)

Edited by Arpit Gogia

Merge request reports