googleapis/mcp-toolbox · error

range %q has a bad suffix length

Error message

range %q has a bad suffix length

What it means

Range-parser guard for the suffix form 'bytes=-N': N is not a parseable integer or is <= 0, so the 'last N bytes' request is meaningless.

Source

Thrown at internal/tools/cloudstorage/cloudstoragereadobject/cloudstoragereadobject.go:184

	}
	spec = strings.TrimPrefix(spec, "bytes=")

	dash := strings.IndexByte(spec, '-')
	if dash < 0 {
		return 0, 0, fmt.Errorf("range %q is missing '-'", rangeSpec)
	}

	startStr := spec[:dash]
	endStr := spec[dash+1:]

	switch {
	case startStr == "" && endStr == "":
		return 0, 0, fmt.Errorf("range %q requires at least one bound", rangeSpec)

	case startStr == "":
		n, parseErr := strconv.ParseInt(endStr, 10, 64)
		if parseErr != nil || n <= 0 {
			return 0, 0, fmt.Errorf("range %q has a bad suffix length", rangeSpec)
		}
		return -n, -1, nil

	case endStr == "":
		start, parseErr := strconv.ParseInt(startStr, 10, 64)
		if parseErr != nil || start < 0 {
			return 0, 0, fmt.Errorf("range %q has a bad start", rangeSpec)
		}
		return start, -1, nil

	default:
		start, parseErr := strconv.ParseInt(startStr, 10, 64)
		if parseErr != nil || start < 0 {
			return 0, 0, fmt.Errorf("range %q has a bad start", rangeSpec)
		}
		end, parseErr := strconv.ParseInt(endStr, 10, 64)
		if parseErr != nil || end < start {
			return 0, 0, fmt.Errorf("range %q has a bad end", rangeSpec)

View on GitHub (pinned to 8cc6e09de2)

Solutions

  1. Use a positive integer suffix length, e.g. 'bytes=-512'
  2. Use an explicit start-based range if the tail length is not what you want
Defensive patterns

Strategy: validation

When it happens

Trigger: Thrown at internal/tools/cloudstorage/cloudstoragereadobject/cloudstoragereadobject.go:184 when the library encounters an invalid state.

Common situations: See trigger scenarios.


AI-assisted analysis of googleapis/mcp-toolbox@8cc6e09de2 (2026-09-05). Data as JSON: /api/errors/bfebd54b7b15869f. Report an issue: GitHub.