Merge pull request #4 from bertrandbrasseur/master

aldycool · web-flow · commit 7f0079b82c49 · 2022-05-23T08:10:18.000-07:00
fix issue with .NET6 (failing decompress large gzip text)
diff --git a/NETCoreSync/SyncEngineHelpers.cs b/NETCoreSync/SyncEngineHelpers.cs
@@ -47,7 +47,13 @@ internal static string Decompress(byte[] data)
             {
                 using (var gzip = new GZipStream(ms, CompressionMode.Decompress))
                 {
-                    gzip.Read(buffer, 0, uncompressedSize);
+                    int totalRead = 0;
+                    while (totalRead < buffer.Length)
+                    {
+                        int bytesRead = gzip.Read(buffer, totalRead, buffer.Length - totalRead);
+                        if (bytesRead == 0) break;
+                        totalRead += bytesRead;
+                    }
                 }
             }
             return Encoding.Unicode.GetString(buffer);

Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,13 @@ internal static string Decompress(byte[] data)`
`47`	`47`	`{`
`48`	`48`	`using (var gzip = new GZipStream(ms, CompressionMode.Decompress))`
`49`	`49`	`{`
`50`		`- gzip.Read(buffer, 0, uncompressedSize);`
	`50`	`+ int totalRead = 0;`
	`51`	`+ while (totalRead < buffer.Length)`
	`52`	`+ {`
	`53`	`+ int bytesRead = gzip.Read(buffer, totalRead, buffer.Length - totalRead);`
	`54`	`+ if (bytesRead == 0) break;`
	`55`	`+ totalRead += bytesRead;`
	`56`	`+ }`
`51`	`57`	`}`
`52`	`58`	`}`
`53`	`59`	`return Encoding.Unicode.GetString(buffer);`