Alter Content.SourceURL length

Author: Fosol

libs/net/dal/Configuration/ContentConfiguration.cs

@@ -29,7 +29,7 @@ public override void Configure(EntityTypeBuilder<Content> builder)
         builder.Property(m => m.IsHidden).IsRequired();
         builder.Property(m => m.IsApproved).IsRequired();
         builder.Property(m => m.IsPrivate).IsRequired();
-        builder.Property(m => m.SourceUrl).IsRequired().HasMaxLength(500);
+        builder.Property(m => m.SourceUrl).IsRequired().HasMaxLength(1000);
         builder.Property(m => m.Headline).IsRequired().HasMaxLength(500);
         builder.Property(m => m.Page).IsRequired().HasMaxLength(20);
         builder.Property(m => m.Summary).IsRequired().HasColumnType("text");

libs/net/dal/Migrations/20250407151823_1.3.25.Designer.cs

@@ -0,0 +1,43 @@
+using Microsoft.EntityFrameworkCore.Migrations;
+using TNO.DAL;
+
+#nullable disable
+
+namespace TNO.DAL.Migrations
+{
+    /// <inheritdoc />
+    public partial class _1325 : SeedMigration
+    {
+        /// <inheritdoc />
+        protected override void Up(MigrationBuilder migrationBuilder)
+        {
+            PreUp(migrationBuilder);
+            migrationBuilder.AlterColumn<string>(
+                name: "source_url",
+                table: "content",
+                type: "character varying(1000)",
+                maxLength: 1000,
+                nullable: false,
+                oldClrType: typeof(string),
+                oldType: "character varying(500)",
+                oldMaxLength: 500);
+            PostUp(migrationBuilder);
+        }
+
+        /// <inheritdoc />
+        protected override void Down(MigrationBuilder migrationBuilder)
+        {
+            PreDown(migrationBuilder);
+            migrationBuilder.AlterColumn<string>(
+                name: "source_url",
+                table: "content",
+                type: "character varying(500)",
+                maxLength: 500,
+                nullable: false,
+                oldClrType: typeof(string),
+                oldType: "character varying(1000)",
+                oldMaxLength: 1000);
+            PostDown(migrationBuilder);
+        }
+    }
+}

libs/net/dal/Migrations/20250407151823_1.3.25.cs

@@ -1040,8 +1040,8 @@ protected override void BuildModel(ModelBuilder modelBuilder)
 
                     b.Property<string>("SourceUrl")
                         .IsRequired()
-                        .HasMaxLength(500)
-                        .HasColumnType("character varying(500)")
+                        .HasMaxLength(1000)
+                        .HasColumnType("character varying(1000)")
                         .HasColumnName("source_url");
 
                     b.Property<int>("Status")

libs/net/dal/Migrations/TNOContextModelSnapshot.cs

@@ -153,7 +153,6 @@ public class ContentModel : AuditColumnsModel
     /// <summary>
     /// get/set - The source URL.
     /// </summary>
-    [MaxLength(500)]
     public string SourceUrl { get; set; } = "";
 
     /// <summary>

libs/net/models/Areas/Editor/Models/Content/ContentModel.cs

@@ -149,7 +149,6 @@ public class ContentModel : AuditColumnsModel
     /// <summary>
     /// get/set - The source URL.
     /// </summary>
-    [MaxLength(500)]
     public string SourceUrl { get; set; } = "";
 
     /// <summary>

libs/net/models/Areas/Services/Models/Content/ContentModel.cs

@@ -63,6 +63,7 @@ spec:
         git checkout $(params.GIT_SOURCE)
 
         # TODO: Make the branch equal to the GIT_REF
+        # git reset --hard origin/master
         git pull
         git show -s
 

openshift/kustomize/tekton/base/tasks/git-conditions.yaml

@@ -548,6 +548,8 @@ private async Task ProcessSourceContentAsync(ConsumeResult<string, SourceContent
                 model.MediaTypeId = ingests?.MediaTypeId ?? throw new InvalidOperationException($"Unable to find an ingest for the topic '{result.Topic}'");
             }
 
+            if (content?.SourceUrl.Length > 1000) this.Logger.LogWarning("Content.SourceUrl is greater than maximum length. UID={uid}", content.Uid);
+
             content ??= new ContentModel();
             content.Uid = model.Uid;
             content.Status = model.Status;
@@ -564,7 +566,7 @@ private async Task ProcessSourceContentAsync(ConsumeResult<string, SourceContent
             content.Summary = String.IsNullOrWhiteSpace(summary) ? "" : summary;
             content.Body = !String.IsNullOrWhiteSpace(body) ? body : model.ContentType == ContentType.AudioVideo ? "" : summary;
             content.IsApproved = model.ContentType == ContentType.AudioVideo && model.PublishedOn.HasValue && model.Status == ContentStatus.Publish && !String.IsNullOrWhiteSpace(model.Body);
-            content.SourceUrl = model.Link;
+            content.SourceUrl = model.Link.Length > 1000 ? model.Link[..1000] : model.Link;
             content.PublishedOn = model.PublishedOn;
             content.Section = model.Section;
             content.Byline = string.Join(",", model.Authors.Select(a => a.Name[0..Math.Min(a.Name.Length, 200)])); // TODO: Temporary workaround to deal with regression issue in Syndication Service.