diff --git a/routers/api/v1/repo/migrate.go b/routers/api/v1/repo/migrate.go
index 17259dc724..9355177fce 100644
--- a/routers/api/v1/repo/migrate.go
+++ b/routers/api/v1/repo/migrate.go
@@ -140,6 +140,7 @@ func Migrate(ctx *context.APIContext) {
 	}
 
 	opts := migrations.MigrateOptions{
+		OriginalURL:    form.CloneAddr,
 		CloneAddr:      remoteAddr,
 		RepoName:       form.RepoName,
 		Description:    form.Description,
diff --git a/services/migrations/migrate.go b/services/migrations/migrate.go
index dfd56144f6..99f8dba92f 100644
--- a/services/migrations/migrate.go
+++ b/services/migrations/migrate.go
@@ -131,9 +131,6 @@ func MigrateRepository(ctx context.Context, doer *user_model.User, ownerName str
 		if err1 := uploader.Rollback(); err1 != nil {
 			log.Error("rollback failed: %v", err1)
 		}
-		// TODO: opts.OriginalURL can be empty when migration is triggered via API (routers/api/v1/repo/migrate.go doesn't set it).
-		// This may result in a message like "Migrate repository (owner/repo) from  failed" when OriginalURL is empty.
-		// Consider either: 1) ensuring OriginalURL is always set in the API, or 2) conditionally formatting this message.
 		if err2 := system_model.CreateRepositoryNotice(fmt.Sprintf("Migrate repository (%s/%s) from %s failed: %v", ownerName, opts.RepoName, opts.OriginalURL, err)); err2 != nil {
 			log.Error("create repository notice failed: ", err2)
 		}