Consumer documentation says it defaults to 5s timeout, but uses a 5m by default

[Zambito1]

Observed behavior

See line 610

nats.go/jetstream/ordered.go

Lines 605 to 615 in da7160e

	cfg := &ConsumerConfig{
	Name: name,
	DeliverPolicy: DeliverByStartSequencePolicy,
	OptStartSeq: nextSeq,
	AckPolicy: AckNonePolicy,
	InactiveThreshold: 5 * time.Minute,
	Replicas: 1,
	HeadersOnly: c.cfg.HeadersOnly,
	MemoryStorage: true,
	Metadata: c.cfg.Metadata,
	}

Expected behavior

See line 195

nats.go/jetstream/consumer_config.go

Lines 190 to 201 in da7160e

	// InactiveThreshold is a duration which instructs the server to clean
	// up the consumer if it has been inactive for the specified duration.
	// Durable consumers will not be cleaned up by default, but if
	// InactiveThreshold is set, they will be. If not set, this will inherit
	// settings from stream's ConsumerLimits. If neither are set, server
	// default is 5 seconds.
	//
	// A consumer is considered inactive there are not pull requests
	// received by the server (for pull consumers), or no interest detected
	// on deliver subject (for push consumers), not if there are no
	// messages to be delivered.
	InactiveThreshold time.Duration `json:"inactive_threshold,omitempty"`

See line 256

nats.go/jetstream/consumer_config.go

Lines 254 to 257 in da7160e

	// InactiveThreshold is a duration which instructs the server to clean
	// up the consumer if it has been inactive for the specified duration.
	// Defaults to 5s.
	InactiveThreshold time.Duration `json:"inactive_threshold,omitempty"`

The docs and the behavior should line up

Server and client version

Client: da7160e

Host environment

N/A

Steps to reproduce

N/A