Configurable DataBag Size/Depth

[s1lenced7]

Hi,

I have a short question concerning some serialisation behaviour.
I noticed that event elements can be truncated in accordance with the following constants:

• MAX_DATABAG_DEPTH
  

  sentry-python/sentry_sdk/serializer.py

  Line 71 in 76aa189

  MAX_DATABAG_DEPTH = 5

• MAX_DATABAG_BREADTH
  

  sentry-python/sentry_sdk/serializer.py

  Line 72 in 76aa189

  MAX_DATABAG_BREADTH = 10

Our implementation requires larger sizes and we've had to manually import the serialisation file and change the value as follows (example):

from sentry_sdk import serializer
serializer.MAX_DATABAG_BREADTH = 50

Question:
Is this the intended way of influencing this behaviour, did we miss a certain configuration for this?

Suggestion:
If this is the intended way perhaps provide a kwarg at sentry_sdk.init instead?

Thx in advance!

[github-actions]

This issue has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you label it Status: Backlog or Status: In Progress, I will leave it alone ... forever!

---

"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀

[antonpirker]

Hello @s1lenced7 sorry for taking soo long to answer your question.

We do not have currently an API to change MAX_DATABAG_DEPTH, so the way you do it is totally fine.
(That`s why we love Python)

Currently we also have no plans on adding this API to the SDK because we want the surface of the SDK be as small as possible.

Hope this answers your question.

[s1lenced7]

Hi @antonpirker,

I completely understand, keeping the surface minimal does decrease the opportunities for user error etc.
I do suggest, if not done so already, to add a section to the documentation on this behaviour.
Or is it preferred to keep this "feature manipulation" out of common use?

Thx for your time and consideration.

[antonpirker]

I think it is OK if people do this kind of "feature manipulation" as you called it. But we will not advertise this in the docs.

Thanks for all your input @s1lenced7 !

[realkosty]

would be nice for SDK to report truncated content and then have UI indicate it was truncated
e.g. (first 10 of total 284)
otherwise people may be confused and think something's wrong in their application state