Skip to content

Fix null ptr with seralizing EnvironBean#1924

Merged
osoriano merged 1 commit intomasterfrom
osoriano/fix-null-ptr
Mar 27, 2026
Merged

Fix null ptr with seralizing EnvironBean#1924
osoriano merged 1 commit intomasterfrom
osoriano/fix-null-ptr

Conversation

@osoriano
Copy link
Copy Markdown
Contributor

@osoriano osoriano commented Mar 27, 2026

Summary

In the previous PR (#1923), Teletraan now sends notifications on stage creation/deletion. However, the jackson mapper throws a null pointer exception since the notify_authors getter/setter uses a primitive value, but the underlying value uses a wrapper (e.g. Boolean instead of boolean).

The notify_authors is the only field affected. Move the getter/setter to the wrapped type.

Testing Done

  • Verified the null pointer exception was being thrown in the dev environment
  • Verified after apply the change, the notification was actually sent

@github-actions github-actions bot added the deploy-service Includes changes to deploy-service label Mar 27, 2026
@osoriano osoriano changed the title fix null ptr with seralizing environbean Fix null ptr with seralizing EnvironBean Mar 27, 2026
@osoriano osoriano marked this pull request as ready for review March 27, 2026 22:29
@osoriano osoriano requested a review from a team as a code owner March 27, 2026 22:29
@osoriano osoriano merged commit 0e953f8 into master Mar 27, 2026
6 checks passed
@osoriano osoriano deleted the osoriano/fix-null-ptr branch March 27, 2026 23:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

deploy-service Includes changes to deploy-service

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants