v3.236.14:

- Fixed GetResourceGroupByFullPathAsync to handle full paths containing parentheses by falling back to tree node search
- Removed EscapeParens() from GetResourceGroupByFullPathAsync which caused double-encoding via HttpUtility.UrlEncode

Author: rolandbanks

LogicMonitor.Api.Test/Resources/ResourceGroupTests.cs

@@ -4,6 +4,13 @@ namespace LogicMonitor.Api.Test.Resources;
 
 public class ResourceGroupTests(ITestOutputHelper iTestOutputHelper, Fixture fixture) : TestWithOutput(iTestOutputHelper, fixture), IClassFixture<Fixture>
 {
+	//[Fact]
+	//public async Task GetResourceGroupByDisplayNameContainsParenthesisAsync()
+	//{
+	//	var x = await LogicMonitorClient.GetResourceGroupByFullPathAsync("DSW - (Test) Denise Home Network", CancellationToken);
+	//	x.Should().NotBeNull();
+	//}
+
 	[Fact]
 	public async Task GetResourceGroupByFullPath()
 	{

LogicMonitor.Api/LogicMonitor.Api.csproj

@@ -24,11 +24,9 @@
     <SymbolPackageFormat>snupkg</SymbolPackageFormat>
     <!-- Update the following before releasing to nuget -->
     <PackageReleaseNotes>
-		v3.236.10:
-		- fix: handle parentheses in GetResourceByDisplayNameAsync
-		The LM API Eq filter no longer accepts parentheses in values.
-		Fall back to tree node search (POST /functions) when the display
-		name contains '(' or ')' and match client-side with SingleOrDefault.
+		v3.236.14:
+		- Fixed GetResourceGroupByFullPathAsync to handle full paths containing parentheses by falling back to tree node search
+		- Removed EscapeParens() from GetResourceGroupByFullPathAsync which caused double-encoding via HttpUtility.UrlEncode
 	</PackageReleaseNotes>
     <UserSecretsId>57aaa0e7-815d-4065-9339-f3f070bed01e</UserSecretsId>
     <PackageId>LogicMonitor.Api</PackageId>

LogicMonitor.Api/LogicMonitorClient_Resources.cs

@@ -408,19 +408,41 @@ public async Task<List<Resource>> GetResourcesByNameAsync(
 			return await GetAsync<ResourceGroup>(1, cancellationToken).ConfigureAwait(false);
 		}
 
-		// This may actually return more than one, as LogicMonitor does not correctly handle brackets in some cases.
-		var allResourceGroups = await GetAllAsync(new Filter<ResourceGroup>
+		// The LM API Eq filter cannot handle parentheses in values - fall back to tree node search
+		if (resourceGroupFullPath.Contains('(') || resourceGroupFullPath.Contains(')'))
 		{
-			FilterItems =
-			[
-				new Eq<ResourceGroup>(nameof(ResourceGroup.FullPath), resourceGroupFullPath.EscapePlusCharacter().EscapeParens())
-			]
-		},
-		cancellationToken: cancellationToken)
-		.ConfigureAwait(false);
+			// Search by leaf group name, then match by full path client-side
+			var leafName = resourceGroupFullPath.Split('/')[^1];
+			var searchResults = await TreeNodeFreeSearchAsync(
+				leafName,
+				100,
+				cancellationToken,
+				TreeNodeFreeSearchResultType.ResourceGroup)
+				.ConfigureAwait(false);
+
+			foreach (var result in searchResults)
+			{
+				var group = await GetAsync<ResourceGroup>(result.EntityId, cancellationToken).ConfigureAwait(false);
+				if (group.FullPath == resourceGroupFullPath)
+				{
+					return group;
+				}
+			}
+
+			return null;
+		}
+
+		// This may actually return more than one, as LogicMonitor does not correctly handle brackets in some cases.
+		var allResourceGroups =
+			await GetAllAsync(new Filter<ResourceGroup>{
+				FilterItems =
+				[
+					new Eq<ResourceGroup>(nameof(ResourceGroup.FullPath), resourceGroupFullPath.EscapePlusCharacter())
+				]},
+				cancellationToken: cancellationToken)
+			.ConfigureAwait(false);
 
-		return allResourceGroups
-			.SingleOrDefault(dg => dg.FullPath == resourceGroupFullPath);
+		return allResourceGroups.SingleOrDefault(dg => dg.FullPath == resourceGroupFullPath);
 	}
 
 	/// <summary>