From 6f4cb2f14e37dc4142ec067f7dc2ed898bf804c2 Mon Sep 17 00:00:00 2001 From: kim <89579420+NyaaaWhatsUpDoc@users.noreply.github.com> Date: Tue, 5 Nov 2024 22:16:06 +0000 Subject: [bugfix] sets the max value placeholders to MaxInt32 instead of MaxInt (#3517) * sets the max value placeholders to MaxInt32 instead of MaxInt * update tests --- internal/typeutils/internaltofrontend.go | 12 ++++++------ internal/typeutils/internaltofrontend_test.go | 12 ++++++------ 2 files changed, 12 insertions(+), 12 deletions(-) (limited to 'internal/typeutils') diff --git a/internal/typeutils/internaltofrontend.go b/internal/typeutils/internaltofrontend.go index 2a6d495d7..b45e0f814 100644 --- a/internal/typeutils/internaltofrontend.go +++ b/internal/typeutils/internaltofrontend.go @@ -1576,9 +1576,9 @@ func (c *Converter) InstanceToAPIV1Instance(ctx context.Context, i *gtsmodel.Ins instance.Configuration.MediaAttachments.VideoSizeLimit = int(videoSz) // #nosec G115 -- Already validated. // we don't actually set any limits on these. set to max possible. - instance.Configuration.MediaAttachments.ImageMatrixLimit = math.MaxInt - instance.Configuration.MediaAttachments.VideoFrameRateLimit = math.MaxInt - instance.Configuration.MediaAttachments.VideoMatrixLimit = math.MaxInt + instance.Configuration.MediaAttachments.ImageMatrixLimit = math.MaxInt32 + instance.Configuration.MediaAttachments.VideoFrameRateLimit = math.MaxInt32 + instance.Configuration.MediaAttachments.VideoMatrixLimit = math.MaxInt32 instance.Configuration.Polls.MaxOptions = config.GetStatusesPollMaxOptions() instance.Configuration.Polls.MaxCharactersPerOption = config.GetStatusesPollOptionMaxChars() @@ -1739,9 +1739,9 @@ func (c *Converter) InstanceToAPIV2Instance(ctx context.Context, i *gtsmodel.Ins instance.Configuration.MediaAttachments.VideoSizeLimit = int(videoSz) // #nosec G115 -- Already validated. // we don't actually set any limits on these. set to max possible. - instance.Configuration.MediaAttachments.ImageMatrixLimit = math.MaxInt - instance.Configuration.MediaAttachments.VideoFrameRateLimit = math.MaxInt - instance.Configuration.MediaAttachments.VideoMatrixLimit = math.MaxInt + instance.Configuration.MediaAttachments.ImageMatrixLimit = math.MaxInt32 + instance.Configuration.MediaAttachments.VideoFrameRateLimit = math.MaxInt32 + instance.Configuration.MediaAttachments.VideoMatrixLimit = math.MaxInt32 instance.Configuration.Polls.MaxOptions = config.GetStatusesPollMaxOptions() instance.Configuration.Polls.MaxCharactersPerOption = config.GetStatusesPollOptionMaxChars() diff --git a/internal/typeutils/internaltofrontend_test.go b/internal/typeutils/internaltofrontend_test.go index 6c318e851..b9cadb183 100644 --- a/internal/typeutils/internaltofrontend_test.go +++ b/internal/typeutils/internaltofrontend_test.go @@ -1968,10 +1968,10 @@ func (suite *InternalToFrontendTestSuite) TestInstanceV1ToFrontend() { "video/x-matroska" ], "image_size_limit": 41943040, - "image_matrix_limit": 9223372036854775807, + "image_matrix_limit": 2147483647, "video_size_limit": 41943040, - "video_frame_rate_limit": 9223372036854775807, - "video_matrix_limit": 9223372036854775807 + "video_frame_rate_limit": 2147483647, + "video_matrix_limit": 2147483647 }, "polls": { "max_options": 6, @@ -2113,10 +2113,10 @@ func (suite *InternalToFrontendTestSuite) TestInstanceV2ToFrontend() { "video/x-matroska" ], "image_size_limit": 41943040, - "image_matrix_limit": 9223372036854775807, + "image_matrix_limit": 2147483647, "video_size_limit": 41943040, - "video_frame_rate_limit": 9223372036854775807, - "video_matrix_limit": 9223372036854775807 + "video_frame_rate_limit": 2147483647, + "video_matrix_limit": 2147483647 }, "polls": { "max_options": 6, -- cgit v1.2.3