Correct cursor parameter in GetFollowers, GetBlocks, GetFollows, GetKnownFollowers, GetListBlocks, GetListMutes, GetList, GetLists, ListConversations. Fixes #165

blowdart · blowdart · commit d625c1966425 · 2025-05-30T18:02:27.000-07:00
diff --git a/history.md b/history.md
@@ -1,5 +1,13 @@
 # Version History
 
+## 0.9.4
+
+### Bug fixes
+
+#### idunno.Bluesky
+
+* Fix cursor pagination in GetFollowers, GetBlocks, GetFollows, GetKnownFollowers, GetListBlocks, GetListMutes, GetList, GetLists, ListConversations.
+
 ## 0.9.3
 
 ### Bug fixes
diff --git a/src/idunno.Bluesky/Chat/BlueskyServer.cs b/src/idunno.Bluesky/Chat/BlueskyServer.cs
@@ -386,7 +386,7 @@ public static async Task<AtProtoHttpResult<Conversations>> ListConversations(
 
             if (cursor is not null)
             {
-                queryStringBuilder.Append(CultureInfo.InvariantCulture, $"&cursor = {Uri.EscapeDataString(cursor)}");
+                queryStringBuilder.Append(CultureInfo.InvariantCulture, $"&cursor={Uri.EscapeDataString(cursor)}");
             }
 
             string queryString = queryStringBuilder.ToString();
diff --git a/src/idunno.Bluesky/Graph/BlueskyServer.cs b/src/idunno.Bluesky/Graph/BlueskyServer.cs
@@ -125,7 +125,7 @@ public static async Task<AtProtoHttpResult<PagedViewReadOnlyCollection<ProfileVi
 
             if (cursor is not null)
             {
-                queryStringBuilder.Append(CultureInfo.InvariantCulture, $"&cursor = {Uri.EscapeDataString(cursor)}");
+                queryStringBuilder.Append(CultureInfo.InvariantCulture, $"&cursor={Uri.EscapeDataString(cursor)}");
             }
 
             string queryString = queryStringBuilder.ToString();
@@ -219,7 +219,7 @@ public static async Task<AtProtoHttpResult<Followers>> GetFollowers(
 
             if (cursor is not null)
             {
-                queryStringBuilder.Append(CultureInfo.InvariantCulture, $"&cursor = {Uri.EscapeDataString(cursor)}");
+                queryStringBuilder.Append(CultureInfo.InvariantCulture, $"&cursor={Uri.EscapeDataString(cursor)}");
             }
 
             string queryString = queryStringBuilder.ToString();
@@ -313,7 +313,7 @@ public static async Task<AtProtoHttpResult<Follows>> GetFollows(
 
             if (cursor is not null)
             {
-                queryStringBuilder.Append(CultureInfo.InvariantCulture, $"&cursor = {Uri.EscapeDataString(cursor)}");
+                queryStringBuilder.Append(CultureInfo.InvariantCulture, $"&cursor={Uri.EscapeDataString(cursor)}");
             }
 
             string queryString = queryStringBuilder.ToString();
@@ -408,7 +408,7 @@ public static async Task<AtProtoHttpResult<Followers>> GetKnownFollowers(
 
             if (cursor is not null)
             {
-                queryStringBuilder.Append(CultureInfo.InvariantCulture, $"&cursor = {Uri.EscapeDataString(cursor)}");
+                queryStringBuilder.Append(CultureInfo.InvariantCulture, $"&cursor={Uri.EscapeDataString(cursor)}");
             }
 
             string queryString = queryStringBuilder.ToString();
@@ -498,7 +498,7 @@ public static async Task<AtProtoHttpResult<PagedViewReadOnlyCollection<ListView>
 
             if (cursor is not null)
             {
-                queryStringBuilder.Append(CultureInfo.InvariantCulture, $"&cursor = {Uri.EscapeDataString(cursor)}");
+                queryStringBuilder.Append(CultureInfo.InvariantCulture, $"&cursor={Uri.EscapeDataString(cursor)}");
             }
 
             string queryString = queryStringBuilder.ToString();
@@ -585,7 +585,7 @@ public static async Task<AtProtoHttpResult<PagedViewReadOnlyCollection<ListView>
 
             if (cursor is not null)
             {
-                queryStringBuilder.Append(CultureInfo.InvariantCulture, $"&cursor = {Uri.EscapeDataString(cursor)}");
+                queryStringBuilder.Append(CultureInfo.InvariantCulture, $"&cursor={Uri.EscapeDataString(cursor)}");
             }
 
             string queryString = queryStringBuilder.ToString();
@@ -677,7 +677,7 @@ public static async Task<AtProtoHttpResult<ListViewWithItems>> GetList(
 
             if (cursor is not null)
             {
-                queryStringBuilder.Append(CultureInfo.InvariantCulture, $"&cursor = {Uri.EscapeDataString(cursor)}");
+                queryStringBuilder.Append(CultureInfo.InvariantCulture, $"&cursor={Uri.EscapeDataString(cursor)}");
             }
 
             string queryString = queryStringBuilder.ToString();
@@ -768,7 +768,7 @@ public static async Task<AtProtoHttpResult<PagedViewReadOnlyCollection<ListView>
 
             if (cursor is not null)
             {
-                queryStringBuilder.Append(CultureInfo.InvariantCulture, $"&cursor = {Uri.EscapeDataString(cursor)}");
+                queryStringBuilder.Append(CultureInfo.InvariantCulture, $"&cursor={Uri.EscapeDataString(cursor)}");
             }
 
             string queryString = queryStringBuilder.ToString();
diff --git a/version.json b/version.json
@@ -1,6 +1,6 @@
 {
     "$schema": "https://raw.githubusercontent.com/AArnott/Nerdbank.GitVersioning/master/src/NerdBank.GitVersioning/version.schema.json",
-    "version": "0.9.3-prerelease",
+    "version": "0.9.4-prerelease",
     "publicReleaseRefSpec": [
         "^refs\/heads\/rel\/.*$"
     ],

Original file line number	Diff line number	Diff line change
`@@ -386,7 +386,7 @@ public static async Task<AtProtoHttpResult<Conversations>> ListConversations(`
`386`	`386`
`387`	`387`	`if (cursor is not null)`
`388`	`388`	`{`
`389`		`- queryStringBuilder.Append(CultureInfo.InvariantCulture, $"&cursor = {Uri.EscapeDataString(cursor)}");`
	`389`	`+ queryStringBuilder.Append(CultureInfo.InvariantCulture, $"&cursor={Uri.EscapeDataString(cursor)}");`
`390`	`390`	`}`
`391`	`391`
`392`	`392`	`string queryString = queryStringBuilder.ToString();`
Original file line number	Diff line number	Diff line change
`@@ -125,7 +125,7 @@ public static async Task<AtProtoHttpResult<PagedViewReadOnlyCollection<ProfileVi`
`125`	`125`
`126`	`126`	`if (cursor is not null)`
`127`	`127`	`{`
`128`		`- queryStringBuilder.Append(CultureInfo.InvariantCulture, $"&cursor = {Uri.EscapeDataString(cursor)}");`
	`128`	`+ queryStringBuilder.Append(CultureInfo.InvariantCulture, $"&cursor={Uri.EscapeDataString(cursor)}");`
`129`	`129`	`}`
`130`	`130`
`131`	`131`	`string queryString = queryStringBuilder.ToString();`
`@@ -219,7 +219,7 @@ public static async Task<AtProtoHttpResult<Followers>> GetFollowers(`
`219`	`219`
`220`	`220`	`if (cursor is not null)`
`221`	`221`	`{`
`222`		`- queryStringBuilder.Append(CultureInfo.InvariantCulture, $"&cursor = {Uri.EscapeDataString(cursor)}");`
	`222`	`+ queryStringBuilder.Append(CultureInfo.InvariantCulture, $"&cursor={Uri.EscapeDataString(cursor)}");`
`223`	`223`	`}`
`224`	`224`
`225`	`225`	`string queryString = queryStringBuilder.ToString();`
`@@ -313,7 +313,7 @@ public static async Task<AtProtoHttpResult<Follows>> GetFollows(`
`313`	`313`
`314`	`314`	`if (cursor is not null)`
`315`	`315`	`{`
`316`		`- queryStringBuilder.Append(CultureInfo.InvariantCulture, $"&cursor = {Uri.EscapeDataString(cursor)}");`
	`316`	`+ queryStringBuilder.Append(CultureInfo.InvariantCulture, $"&cursor={Uri.EscapeDataString(cursor)}");`
`317`	`317`	`}`
`318`	`318`
`319`	`319`	`string queryString = queryStringBuilder.ToString();`
`@@ -408,7 +408,7 @@ public static async Task<AtProtoHttpResult<Followers>> GetKnownFollowers(`
`408`	`408`
`409`	`409`	`if (cursor is not null)`
`410`	`410`	`{`
`411`		`- queryStringBuilder.Append(CultureInfo.InvariantCulture, $"&cursor = {Uri.EscapeDataString(cursor)}");`
	`411`	`+ queryStringBuilder.Append(CultureInfo.InvariantCulture, $"&cursor={Uri.EscapeDataString(cursor)}");`
`412`	`412`	`}`
`413`	`413`
`414`	`414`	`string queryString = queryStringBuilder.ToString();`
`@@ -498,7 +498,7 @@ public static async Task<AtProtoHttpResult<PagedViewReadOnlyCollection<ListView>`
`498`	`498`
`499`	`499`	`if (cursor is not null)`
`500`	`500`	`{`
`501`		`- queryStringBuilder.Append(CultureInfo.InvariantCulture, $"&cursor = {Uri.EscapeDataString(cursor)}");`
	`501`	`+ queryStringBuilder.Append(CultureInfo.InvariantCulture, $"&cursor={Uri.EscapeDataString(cursor)}");`
`502`	`502`	`}`
`503`	`503`
`504`	`504`	`string queryString = queryStringBuilder.ToString();`
`@@ -585,7 +585,7 @@ public static async Task<AtProtoHttpResult<PagedViewReadOnlyCollection<ListView>`
`585`	`585`
`586`	`586`	`if (cursor is not null)`
`587`	`587`	`{`
`588`		`- queryStringBuilder.Append(CultureInfo.InvariantCulture, $"&cursor = {Uri.EscapeDataString(cursor)}");`
	`588`	`+ queryStringBuilder.Append(CultureInfo.InvariantCulture, $"&cursor={Uri.EscapeDataString(cursor)}");`
`589`	`589`	`}`
`590`	`590`
`591`	`591`	`string queryString = queryStringBuilder.ToString();`
`@@ -677,7 +677,7 @@ public static async Task<AtProtoHttpResult<ListViewWithItems>> GetList(`
`677`	`677`
`678`	`678`	`if (cursor is not null)`
`679`	`679`	`{`
`680`		`- queryStringBuilder.Append(CultureInfo.InvariantCulture, $"&cursor = {Uri.EscapeDataString(cursor)}");`
	`680`	`+ queryStringBuilder.Append(CultureInfo.InvariantCulture, $"&cursor={Uri.EscapeDataString(cursor)}");`
`681`	`681`	`}`
`682`	`682`
`683`	`683`	`string queryString = queryStringBuilder.ToString();`
`@@ -768,7 +768,7 @@ public static async Task<AtProtoHttpResult<PagedViewReadOnlyCollection<ListView>`
`768`	`768`
`769`	`769`	`if (cursor is not null)`
`770`	`770`	`{`
`771`		`- queryStringBuilder.Append(CultureInfo.InvariantCulture, $"&cursor = {Uri.EscapeDataString(cursor)}");`
	`771`	`+ queryStringBuilder.Append(CultureInfo.InvariantCulture, $"&cursor={Uri.EscapeDataString(cursor)}");`
`772`	`772`	`}`
`773`	`773`
`774`	`774`	`string queryString = queryStringBuilder.ToString();`
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"$schema": "https://raw.githubusercontent.com/AArnott/Nerdbank.GitVersioning/master/src/NerdBank.GitVersioning/version.schema.json",`
`3`		`- "version": "0.9.3-prerelease",`
	`3`	`+ "version": "0.9.4-prerelease",`
`4`	`4`	`"publicReleaseRefSpec": [`
`5`	`5`	`"^refs\/heads\/rel\/.*$"`
`6`	`6`	`],`