[PR #266] [MERGED] Add check to ensure Range is less than content #471

New issue

Closed

opened 2026-03-03 12:30:10 +03:00 by kerem · 0 comments

kerem commented

2026-03-03 12:30:10 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/fsouza/fake-gcs-server/pull/266
Author: @gjabell
Created: 6/8/2020
Status: ✅ Merged
Merged: 6/8/2020
Merged by: @fsouza

Base: master ← Head: master

📝 Commits (2)

f95861b Add check to ensure Range is less than content
33a5981 Add tests for Range handler

📊 Changes

2 files changed (+46 additions, -0 deletions)

View changed files

📝 fakestorage/object.go (+3 -0)
📝 fakestorage/server_test.go (+43 -0)

📄 Description

This fixes the case where a Range value greater than the length of the
content is requested, causing the server to crash and fail the
request.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/fsouza/fake-gcs-server/pull/266 **Author:** [@gjabell](https://github.com/gjabell) **Created:** 6/8/2020 **Status:** ✅ Merged **Merged:** 6/8/2020 **Merged by:** [@fsouza](https://github.com/fsouza) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (2) - [`f95861b`](https://github.com/fsouza/fake-gcs-server/commit/f95861b0d3ac3917956647cacb19dfbcbd3d6f30) Add check to ensure Range is less than content - [`33a5981`](https://github.com/fsouza/fake-gcs-server/commit/33a59812181ae79b83af87b9f7ce683747c863c4) Add tests for Range handler ### 📊 Changes **2 files changed** (+46 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `fakestorage/object.go` (+3 -0) 📝 `fakestorage/server_test.go` (+43 -0) </details> ### 📄 Description * This fixes the case where a Range value greater than the length of the content is requested, causing the server to crash and fail the request. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>