From b505d88d35e43c49b275157848b5cd879fd6fadc Mon Sep 17 00:00:00 2001
From: Vladimir Pecanac <vladimir.pecanac@gmail.com>
Date: Wed, 19 Aug 2026 13:03:39 +0200
Subject: [PATCH] Update Firebase sample: .NET 10, current bucket domain,
 cleanups

- Retarget Firebase and Tests to net10.0 and bump packages
  (Google.Cloud.Firestore 4.4.0, Google.Cloud.Storage.V1 4.15.0,
  FirebaseAdmin 3.6.0, FirebaseAuthentication.net 4.1.0,
  JwtBearer/Mvc.Testing 10.0.11, test tooling to current).
- BucketName now uses the current default Cloud Storage domain
  PROJECT_ID.firebasestorage.app; appspot.com only applies to
  buckets created before September 2024.
- Drop the redundant [Required] from ShoeDto.Price: on a non-nullable
  decimal the model binder cannot tell "absent" from zero, so the
  attribute implies a guarantee it does not provide.
- Replace the bare <label> elements in the shoe list with plain table
  cells; a label with no for attribute and no associated control is
  meaningless markup and degrades screen-reader output.
---
 .../Firebase/Firebase/Dtos/ShoeDto.cs         |  1 -
 .../Firebase/Firebase/Firebase.csproj         | 12 +++----
 .../Firebase/Pages/Firestore/Index.cshtml     | 36 +++++--------------
 .../Services/FirebaseStorageService.cs        |  2 +-
 .../Firebase/Tests/Tests.csproj               | 15 ++++----
 5 files changed, 24 insertions(+), 42 deletions(-)

diff --git a/dotnet-client-libraries/Firebase/Firebase/Dtos/ShoeDto.cs b/dotnet-client-libraries/Firebase/Firebase/Dtos/ShoeDto.cs
index c5e00dd2ad..3b84bbac32 100644
--- a/dotnet-client-libraries/Firebase/Firebase/Dtos/ShoeDto.cs
+++ b/dotnet-client-libraries/Firebase/Firebase/Dtos/ShoeDto.cs
@@ -10,7 +10,6 @@ public class ShoeDto
     [Required]
     public required string Brand { get; set; }
 
-    [Required]
     public required decimal Price { get; set; }
 
     [Required]
diff --git a/dotnet-client-libraries/Firebase/Firebase/Firebase.csproj b/dotnet-client-libraries/Firebase/Firebase/Firebase.csproj
index bcabba0e3e..b66d4bde5f 100644
--- a/dotnet-client-libraries/Firebase/Firebase/Firebase.csproj
+++ b/dotnet-client-libraries/Firebase/Firebase/Firebase.csproj
@@ -1,17 +1,17 @@
 <Project Sdk="Microsoft.NET.Sdk.Web">
 
   <PropertyGroup>
-    <TargetFramework>net7.0</TargetFramework>
+    <TargetFramework>net10.0</TargetFramework>
     <Nullable>enable</Nullable>
     <ImplicitUsings>enable</ImplicitUsings>
   </PropertyGroup>
 
   <ItemGroup>
-    <PackageReference Include="FirebaseAdmin" Version="2.3.0" />
-    <PackageReference Include="FirebaseAuthentication.net" Version="4.0.2" />
-    <PackageReference Include="Google.Cloud.Firestore" Version="3.3.0" />
-    <PackageReference Include="Google.Cloud.Storage.V1" Version="4.5.0" />
-    <PackageReference Include="Microsoft.AspNetCore.Authentication.JwtBearer" Version="7.0.5" />
+    <PackageReference Include="FirebaseAdmin" Version="3.6.0" />
+    <PackageReference Include="FirebaseAuthentication.net" Version="4.1.0" />
+    <PackageReference Include="Google.Cloud.Firestore" Version="4.4.0" />
+    <PackageReference Include="Google.Cloud.Storage.V1" Version="4.15.0" />
+    <PackageReference Include="Microsoft.AspNetCore.Authentication.JwtBearer" Version="10.0.11" />
   </ItemGroup>
 
 </Project>
diff --git a/dotnet-client-libraries/Firebase/Firebase/Pages/Firestore/Index.cshtml b/dotnet-client-libraries/Firebase/Firebase/Pages/Firestore/Index.cshtml
index 7ca2aaf629..ce032d022e 100644
--- a/dotnet-client-libraries/Firebase/Firebase/Pages/Firestore/Index.cshtml
+++ b/dotnet-client-libraries/Firebase/Firebase/Pages/Firestore/Index.cshtml
@@ -12,39 +12,21 @@
 <table class="table shoe-table">
     <thead>
         <tr>
-            <th>
-                <label>Id</label>
-            </th>
-            <th>
-                <label>Name</label>
-            </th>
-            <th>
-                <label>Brand</label>
-            </th>
-            <th>
-                <label>Price</label>
-            </th>
-            <th>
-                <label>Image</label>
-            </th>
+            <th>Id</th>
+            <th>Name</th>
+            <th>Brand</th>
+            <th>Price</th>
+            <th>Image</th>
         </tr>
     </thead>
     <tbody>
         @foreach (var shoe in Model.Shoes)
         {
             <tr>
-                <td>
-                    <label>@shoe.Id</label>
-                </td>
-                <td>
-                    <label>@shoe.Name</label>
-                </td>
-                <td>
-                    <label>@shoe.Brand</label>
-                </td>
-                <td>
-                    <label>@shoe.Price</label>
-                </td>
+                <td>@shoe.Id</td>
+                <td>@shoe.Name</td>
+                <td>@shoe.Brand</td>
+                <td>@shoe.Price</td>
                 <td>
                     <img src="@shoe.ImageUri" alt="shoe-image">
                 </td>
diff --git a/dotnet-client-libraries/Firebase/Firebase/Services/FirebaseStorageService.cs b/dotnet-client-libraries/Firebase/Firebase/Services/FirebaseStorageService.cs
index bb6aa8ea4b..a230c6f9e0 100644
--- a/dotnet-client-libraries/Firebase/Firebase/Services/FirebaseStorageService.cs
+++ b/dotnet-client-libraries/Firebase/Firebase/Services/FirebaseStorageService.cs
@@ -6,7 +6,7 @@ namespace Firebase.Services;
 public class FirebaseStorageService : IFirebaseStorageService
 {
     private readonly StorageClient _storageClient;
-    private const string BucketName = "codemaze-firebase.appspot.com";
+    private const string BucketName = "codemaze-firebase.firebasestorage.app";
 
     public FirebaseStorageService(StorageClient storageClient)
     {
diff --git a/dotnet-client-libraries/Firebase/Tests/Tests.csproj b/dotnet-client-libraries/Firebase/Tests/Tests.csproj
index 925c5413f7..e4a5ce4d0c 100644
--- a/dotnet-client-libraries/Firebase/Tests/Tests.csproj
+++ b/dotnet-client-libraries/Firebase/Tests/Tests.csproj
@@ -1,23 +1,24 @@
 <Project Sdk="Microsoft.NET.Sdk">
 
   <PropertyGroup>
-    <TargetFramework>net7.0</TargetFramework>
+    <TargetFramework>net10.0</TargetFramework>
     <ImplicitUsings>enable</ImplicitUsings>
     <Nullable>enable</Nullable>
 
     <IsPackable>false</IsPackable>
+    <IsTestProject>true</IsTestProject>
   </PropertyGroup>
 
   <ItemGroup>
-    <PackageReference Include="Microsoft.AspNetCore.Mvc.Testing" Version="7.0.5" />
-    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.3.2" />
-    <PackageReference Include="Moq" Version="4.18.4" />
-    <PackageReference Include="xunit" Version="2.4.2" />
-    <PackageReference Include="xunit.runner.visualstudio" Version="2.4.5">
+    <PackageReference Include="Microsoft.AspNetCore.Mvc.Testing" Version="10.0.11" />
+    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="18.9.0" />
+    <PackageReference Include="Moq" Version="4.20.72" />
+    <PackageReference Include="xunit" Version="2.9.3" />
+    <PackageReference Include="xunit.runner.visualstudio" Version="2.8.2">
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
       <PrivateAssets>all</PrivateAssets>
     </PackageReference>
-    <PackageReference Include="coverlet.collector" Version="3.1.2">
+    <PackageReference Include="coverlet.collector" Version="10.0.1">
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
       <PrivateAssets>all</PrivateAssets>
     </PackageReference>