Skip to content

Conversation

JoshuaMoelans
Copy link
Member

@JoshuaMoelans JoshuaMoelans commented Jul 3, 2025

Related to #1272 and #1271


⚠️ on hold for now, we will first release the initial logs API & keep this as future work


We experiment with an alternative API that looks like this:

sentry_value_t parameters = sentry_value_new_list();
sentry_value_t param = sentry_value_new_object();
sentry_value_set_by_key(param, "name", sentry_value_new_string("intparam"));
sentry_value_set_by_key(param, "value", sentry_value_new_int32(-1));
sentry_value_append(parameters, param);
sentry_log_debug_value("My intparam value is now %i", parameters);

Additionally, to adhere to the spec stating that signed integers should be 64bit, we add a new sentry_value_t for both int64 and uint64.

ToDo

‼️ Unsigned 64 bit integers should always be sent as strings.

#skip-changelog

Copy link

github-actions bot commented Jul 3, 2025

Messages
📖 Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against 66d65fa

@JoshuaMoelans JoshuaMoelans mentioned this pull request Jul 28, 2025
21 tasks
Base automatically changed from joshua/feat/logs_logger_module to joshua/feat/logs July 28, 2025 10:58
Base automatically changed from joshua/feat/logs to master September 23, 2025 13:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant