Merge pull request #987 from DuendeSoftware/mb/fix

Update IdentityServer upgrade guide with improved application name handling

Author: maartenba

.gitignore

@@ -22,4 +22,8 @@ pnpm-debug.log*
 .DS_Store
 .vscode/settings.json
 *.iml
+
+# Agents and assistants
+.opencode/
+opencode.json
 .idea/**/copilot.data.migration.*.xml

src/content/docs/identityserver/upgrades/identityserver4-to-duende-identityserver-v7.mdx

@@ -534,12 +534,20 @@ In your current (pre-upgraded) IdentityServer version, you can query the applica
 var app = builder.Build();
 
 // ...
-var applicationName = app.Services
-    .GetRequiredService<IOptions<DataProtectionOptions>>()
-    .Value.ApplicationDiscriminator;
-
+var applicationName = "";
+var applicationDiscriminatorService = app.Services.GetService<IApplicationDiscriminator>();
+if (applicationDiscriminatorService != null)
+{
+    applicationName = applicationDiscriminatorService.Discriminator;
+}
+var dataProtectionOptions = app.Services.GetService<IOptions<DataProtectionOptions>>();
+if (dataProtectionOptions != null && dataProtectionOptions.Value.ApplicationDiscriminator != null)
+{
+    dataProtectionApplicationDiscriminator = dataProtectionOptions.Value.ApplicationDiscriminator;
+}
+            
 // applicationName is now the (auto-generated?) application name
-// - you can use it in your upgraded IdentityServer version
+// - you can use its value in your upgraded IdentityServer version
 ```
 
 :::