Skip to content
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.

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

fix(nuxt): Detect Azure Function runtime for flushing with timeout #15288

Merged
merged 1 commit into from
Feb 5, 2025

Conversation

s1gr1d
Copy link
Member

@s1gr1d s1gr1d commented Feb 4, 2025

The debugging ability for Azure Functions when using Azure Static Web Apps is very limited. But as I could see some Sentry-related logs I think Sentry is generally initialized.

However, server-related logs don't show up in Sentry. This is probably because the Azure Function finishes before the error can be sent to Sentry.
By adding the Azure environment variable to check whether we should flush with a timeout, this should possibly fix the issue.

Logs in Application Insights:
image

@s1gr1d s1gr1d requested review from Lms24, andreiborza and mydea February 4, 2025 14:31
@s1gr1d s1gr1d merged commit b720e15 into develop Feb 5, 2025
36 checks passed
@s1gr1d s1gr1d deleted the sig/nuxt-azure-serverless branch February 5, 2025 08:05
Zen-cronic pushed a commit to Zen-cronic/sentry-javascript that referenced this pull request Feb 5, 2025
…etsentry#15288)

The debugging ability for Azure Functions when using Azure Static Web
Apps is very limited. But as I could see some Sentry-related logs I
**think** Sentry is generally initialized.

However, server-related logs don't show up in Sentry. This is
**probably** because the Azure Function finishes before the error can be
sent to Sentry.
By adding the Azure environment variable to check whether we should
flush with a timeout, this should possibly fix the issue.

Logs in Application Insights:

![image](https://github.com/user-attachments/assets/ed5ccd37-be87-4338-946b-94ec543e08c5)
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