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

@github-actions
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
@JoshuaMoelans
Copy link
Member Author

closing this for now, until there is interest for this feature on #1398

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.

2 participants