Skip to content

Fix ArgumentError for bulk insert

Saikat Sarkar requested to merge fix_bug_n+1 into master

What does this MR do?

This MR is related to the following issues:

We have some attributes missing in the records while we are performing bulk insert. In the MR, we checked whether we have all the required attributes in the records for bulk insert.

We have some related kibana log: https://gitlab.com/gitlab-org/security/www-gitlab-com/-/issues/12#note_563092020. But, it is not clear which fields are missing during bulk_update.

Screenshots (strongly suggested)

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by Saikat Sarkar

Merge request reports