RequestTooBigException should be ValueTooLargeException

Description

The docs have the wrong exception here: https://docs.couchbase.com/dotnet-sdk/current/howtos/error-handling.html#document-body-too-large

Reported on Gitter.im:

Hey All, I am busy looking at https://docs.couchbase.com/server/current/learn/clusters-and-availability/size-limitations.html and seeing how we could cater for these limits (CouchbaseNetClient query). In the documentation it mentions this https://docs.couchbase.com/dotnet-sdk/current/howtos/error-handling.html#document-body-too-large but this type 'RequestTooBigException' does not exist in https://github.com/couchbase/couchbase-net-client/search?q=RequestTooBigException? potentially the docs are outdated?

With regards to Max Key Length, I noticed there is no validation or specific exception. A consumer would land up receiving 'The output byte buffer is too small to contain the encoded data, encoding 'Unicode (UTF-8)' fallback 'System.Text.EncoderReplacementFallback'. I know it is potentially an edge case but I wanted to ask whether we would consider adding some defensive code in the SDK against this scenario?

Environment

None

Gerrit Reviews

None

Release Notes Description

None

Activity

Show:

Jeffry Morris January 6, 2022 at 12:19 AM

Fixed
Pinned fields
Click on the next to a field label to start pinning.

Details

Assignee

Reporter

Story Points

Components

Fix versions

Affects versions

Priority

Instabug

Open Instabug

PagerDuty

Sentry

Zendesk Support

Created November 13, 2021 at 5:42 AM
Updated January 6, 2022 at 12:20 AM
Resolved January 6, 2022 at 12:20 AM
Instabug