Skip to content

Commit 7396835

Browse files
authored
Tech/warnings (#3117)
* Enable & fix CA1835 * filename
1 parent 922d402 commit 7396835

File tree

3 files changed

+5
-4
lines changed

3 files changed

+5
-4
lines changed

.editorconfig

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -210,3 +210,4 @@ dotnet_style_qualification_for_field = false:warning
210210
dotnet_style_qualification_for_property = false:warning
211211
dotnet_style_qualification_for_method = false:warning
212212
dotnet_style_qualification_for_event = false:warning
213+
dotnet_diagnostic.CA1835.severity = warning

src/JoinRpg.BlobStorage/AvatarDownloader.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ public AvatarDownloader(IHttpClientFactory httpClientFactory, ILogger<AvatarDown
3232
var buffer = ArrayPool<byte>.Shared.Rent(8192);
3333

3434
int read;
35-
while ((read = await stream.ReadAsync(buffer, 0, buffer.Length, ct)) > 0)
35+
while ((read = await stream.ReadAsync(buffer, ct)) > 0)
3636
{
3737
totalRead += read;
3838
if (totalRead > maxSize)

src/JoinRpg.BlobStorage/S3AvatarStorageService.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -23,9 +23,9 @@ internal class S3AvatarStorageService(AvatarDownloader avatarDownloader,
2323
logger.LogInformation("Start uploading avatar from {avatarUri}", remoteUri);
2424

2525
using var memoryStream = new MemoryStream(8192);
26-
var downloadResult = await avatarDownloader.DownloadAvatarAsync(remoteUri, memoryStream, ct);
26+
var (contentType, extension) = await avatarDownloader.DownloadAvatarAsync(remoteUri, memoryStream, ct);
2727

28-
var avatarName = CreateAvatarBlobName(remoteUri, downloadResult.Extension);
28+
var avatarName = CreateAvatarBlobName(remoteUri, extension);
2929

3030
var hash = SHA256.HashData(memoryStream);
3131
memoryStream.Seek(0, SeekOrigin.Begin);
@@ -36,7 +36,7 @@ internal class S3AvatarStorageService(AvatarDownloader avatarDownloader,
3636
BucketName = options.BucketName,
3737
Key = avatarName,
3838
InputStream = memoryStream,
39-
ContentType = downloadResult.ContentType,
39+
ContentType = contentType,
4040
ChecksumSHA256 = Convert.ToBase64String(hash),
4141
};
4242

0 commit comments

Comments
 (0)