Skip to content

Fix null exception message error #46001

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 4 commits into from
Jul 15, 2025
Merged

Fix null exception message error #46001

merged 4 commits into from
Jul 15, 2025
+2 −0

Conversation

trask
Copy link
Member

@trask trask commented Jul 13, 2025 •
edited
Loading

@github-actions github-actions bot added the Monitor - Autoconfigure Monitor OpenTelemetry Autoconfigure label Jul 13, 2025
@trask trask mentioned this pull request Jul 13, 2025
Open
@trask trask marked this pull request as ready for review July 14, 2025 01:39
@Copilot Copilot AI review requested due to automatic review settings July 14, 2025 01:39
Copilot
Copilot AI reviewed Jul 14, 2025
View reviewed changes
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR fixes a null exception message error by ensuring that exception telemetry always has a message value. When the exception message is null or empty, it now falls back to using the exception type as the message to prevent rejection by the Breeze service.

Key Changes:

  • Added fallback logic to use exception type as message when exception message is null/empty
  • Prevents Breeze service from rejecting exceptions without messages

@trask trask merged commit 97c81ab into main Jul 15, 2025
17 checks passed
@trask trask deleted the fix-null-message branch July 15, 2025 04:06
mmoayyed pushed a commit to mmoayyed/cas that referenced this pull request Aug 13, 2025
@apereocas-bot
renovatebot(deps): update dependency com.microsoft.azure:applicationi…
e908373 
…nsights-runtime-attach to v3.7.4

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [com.microsoft.azure:applicationinsights-runtime-attach](https://redirect.github.com/Microsoft/ApplicationInsights-Java) | dependencies | patch | `3.7.3` -> `3.7.4` |

---

> [!WARNING]
> Some dependencies could not be looked up. Check the Dependency Dashboard for more information.

---

### Release Notes

<details>
<summary>Microsoft/ApplicationInsights-Java (com.microsoft.azure:applicationinsights-runtime-attach)</summary>

### [`v3.7.4`](https://redirect.github.com/Microsoft/ApplicationInsights-Java/blob/HEAD/CHANGELOG.md#Version-374-GA-08052025)

##### Bug fixes

- Update deprecated EnvironmentResourceProvider reference
  ([#&#8203;4377](https://redirect.github.com/microsoft/ApplicationInsights-Java/pull/4377))
- Fix null exception message error
  ([Azure/azure-sdk-for-java#46001](https://redirect.github.com/Azure/azure-sdk-for-java/pull/46001))
- Update to OpenTelemetry Java Instrumentation 2.18.1
  ([#&#8203;4378](https://redirect.github.com/microsoft/ApplicationInsights-Java/pull/4378))

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "after 5pm,before 9am" in timezone America/Los_Angeles, Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box

---

This PR has been generated by [Renovate Bot](https://redirect.github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0MS42Ni4xIiwidXBkYXRlZEluVmVyIjoiNDEuNjYuMSIsInRhcmdldEJyYW5jaCI6Im1hc3RlciIsImxhYmVscyI6WyJCb3QiLCJSZW5vdmF0ZSJdfQ==-->
apereocas-bot pushed a commit to apereo/cas that referenced this pull request Aug 13, 2025
@renovate
renovatebot(deps): update dependency com.microsoft.azure:applicationi…
0f0b942 
…nsights-runtime-attach to v3.7.4

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [com.microsoft.azure:applicationinsights-runtime-attach](https://redirect.github.com/Microsoft/ApplicationInsights-Java) | `3.7.3` -> `3.7.4` | [![age](https://developer.mend.io/api/mc/badges/age/maven/com.microsoft.azure:applicationinsights-runtime-attach/3.7.4?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/maven/com.microsoft.azure:applicationinsights-runtime-attach/3.7.4?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/maven/com.microsoft.azure:applicationinsights-runtime-attach/3.7.3/3.7.4?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/maven/com.microsoft.azure:applicationinsights-runtime-attach/3.7.3/3.7.4?slim=true)](https://docs.renovatebot.com/merge-confidence/) |

---

> [!WARNING]
> Some dependencies could not be looked up. Check the Dependency Dashboard for more information.

---

### Release Notes

<details>
<summary>Microsoft/ApplicationInsights-Java (com.microsoft.azure:applicationinsights-runtime-attach)</summary>

### [`v3.7.4`](https://redirect.github.com/Microsoft/ApplicationInsights-Java/blob/HEAD/CHANGELOG.md#Version-374-GA-08052025)

##### Bug fixes

- Update deprecated EnvironmentResourceProvider reference
  ([#&#8203;4377](https://redirect.github.com/microsoft/ApplicationInsights-Java/pull/4377))
- Fix null exception message error
  ([Azure/azure-sdk-for-java#46001](https://redirect.github.com/Azure/azure-sdk-for-java/pull/46001))
- Update to OpenTelemetry Java Instrumentation 2.18.1
  ([#&#8203;4378](https://redirect.github.com/microsoft/ApplicationInsights-Java/pull/4378))

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "after 5pm,before 9am" in timezone America/Los_Angeles, Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box

---

This PR was generated by [Mend Renovate](https://mend.io/renovate/). View the [repository job log](https://developer.mend.io/github/apereo/cas).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0MS42MC40IiwidXBkYXRlZEluVmVyIjoiNDEuNjAuNCIsInRhcmdldEJyYW5jaCI6Im1hc3RlciIsImxhYmVscyI6WyJCb3QiLCJSZW5vdmF0ZSJdfQ==-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

Copilot code review Copilot Copilot left review comments

@xiang17 xiang17 xiang17 approved these changes

@harsimar harsimar harsimar approved these changes

@ramthi ramthi Awaiting requested review from ramthi ramthi is a code owner

@heyams heyams Awaiting requested review from heyams heyams is a code owner

@jeanbisutti jeanbisutti Awaiting requested review from jeanbisutti jeanbisutti is a code owner

@rajkumar-rangaraj rajkumar-rangaraj Awaiting requested review from rajkumar-rangaraj rajkumar-rangaraj is a code owner

Assignees
No one assigned
Labels
Monitor - Autoconfigure Monitor OpenTelemetry Autoconfigure
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Exception without message fails to send to Application Insights
3 participants
@trask @xiang17 @harsimar