Skip to content

Checks if import data contains credentials and data

Madelein van Niekerk requested to merge fix/bitbucket-server-project-data into master

What does this MR do and why?

Fixes sentry error https://new-sentry.gitlab.net/organizations/gitlab/issues/492438:

NoMethodError
 
Sidekiq/Gitlab::BitbucketServerImport::ImportPullRequestNotesWorker

undefined method `[]' for nil:NilClass project.import_data.data['project_key']

This is fixed by checking if both import_data.credentials and import_data.data is present before continuing.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports