Skip to content

Commit

Permalink
Merge pull request #858 from DuendeSoftware/brock/idp-properties-read…
Browse files Browse the repository at this point in the history
…only

Remove setter from Properties on IdentityProvider
  • Loading branch information
leastprivilege authored Apr 19, 2022
2 parents 20b7cc6 + 28311a7 commit 67ba00b
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 2 deletions.
6 changes: 5 additions & 1 deletion migrations/IdentityServerDb/SeedData.cs
Original file line number Diff line number Diff line change
Expand Up @@ -107,7 +107,11 @@ private static void EnsureSeedData(ConfigurationDbContext context)
DisplayName = "Google",
Authority = "https://accounts.google.com",
ClientId = "998042782978-gkes3j509qj26omrh6orvrnu0klpflh6.apps.googleusercontent.com",
Scope = "openid profile email"
Scope = "openid profile email",
Properties =
{
{ "foo", "bar" }
}
}.ToEntity());
context.SaveChanges();
}
Expand Down
2 changes: 1 addition & 1 deletion src/Storage/Models/IdentityProvider.cs
Original file line number Diff line number Diff line change
Expand Up @@ -78,7 +78,7 @@ public IdentityProvider(string type, IdentityProvider other) : this(type)
/// <summary>
/// Protocol specific properties for the provider.
/// </summary>
public Dictionary<string, string> Properties { get; set; } = new Dictionary<string, string>();
public Dictionary<string, string> Properties { get; } = new Dictionary<string, string>();

/// <summary>
/// Properties indexer
Expand Down

0 comments on commit 67ba00b

Please sign in to comment.