Skip to content

Commit 83dc6ad

Browse files
authored
LA-CTPL-1872 - refactor (#23)
* 1. Refactor to more granular, cohisive and single responsibility methods. 2. Retrieve the Auth0 connection, client, and client connection resources from Auth0 on each reconciliation loop. CTPL-1872 * Exclude the A0 client's enabled_connections field from the general client config drift detection as it's not a field retrieved by the get client Auth0 API. CTPL-1872 * Make drift detection logs more prominent. CTPL-1872 * Logging the entity type and name for each drift detected. CTPL-1872 * Extend the retrieval of the current Auth0 state with an extensible (method `RetrieveCurrentAuth0State`), per entity controller, ability to fetch custom entity fields (method `EnrichAuth0State`). CTPL-1872 * fix EnrichWithEnabledConnections CTPL-1872 * Improved logging CTPL-1872 * Amending method header comments. CTPL-1872 * Remove redundant file CTPL-1872 * Handle null warnings CTPL-1872 * Resolve all null warnings. CTPL-1872
1 parent 4033a51 commit 83dc6ad

File tree

10 files changed

+1700
-1003
lines changed

10 files changed

+1700
-1003
lines changed

src/Alethic.Auth0.Operator.Core/Extensions/JTokenExtensions.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,9 +12,9 @@ public static class JTokenExtensions
1212
/// <summary>
1313
/// Copies the <see cref="JToken"/> to an anonymous hierarchy where objects are dictionaries.
1414
/// </summary>
15-
/// <param name="token"></param>
16-
/// <returns></returns>
17-
/// <exception cref="InvalidOperationException"></exception>
15+
/// <param name="token">The JToken to convert to dictionary hierarchy</param>
16+
/// <returns>Dictionary representation of the JToken or null if the token is null</returns>
17+
/// <exception cref="InvalidOperationException">Thrown when the token type is not supported</exception>
1818
public static object? ToDictionary(this JToken token)
1919
{
2020
return token switch

src/Alethic.Auth0.Operator.Tests/ConnectionTests.cs

Lines changed: 0 additions & 33 deletions
This file was deleted.

0 commit comments

Comments
 (0)