-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
[dev-launcher][Android] Send uncaught exceptions to bundler server #15938
[dev-launcher][Android] Send uncaught exceptions to bundler server #15938
Conversation
...uncher/android/src/main/java/expo/modules/devlauncher/helpers/DevLauncherOkHttpExtensions.kt
Outdated
Show resolved
Hide resolved
...rc/main/java/expo/modules/devlauncher/launcher/errors/DevLauncherUncaughtExceptionHandler.kt
Outdated
Show resolved
Hide resolved
...xpo-dev-launcher/android/src/main/java/expo/modules/devlauncher/logs/DevLauncherRemoteLog.kt
Outdated
Show resolved
Hide resolved
...xpo-dev-launcher/android/src/main/java/expo/modules/devlauncher/logs/DevLauncherRemoteLog.kt
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good to me other than the expo-manifests thing, which i think would be fine to fix in a follow-up PR.
} | ||
|
||
private fun getLogsUrl(): Uri { | ||
val logsUrlFromManifest = controller.manifest?.getRawJson()?.optString("logUrl") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this should really go through one of the getters in expo-manifest instead of using getRawJson
(might need to add getLogUrl
there if it doesn't already exist) because this will still only work for classic update manifests.
c9b640d
to
7660d6d
Compare
Hi there! 👋 I'm a bot whose goal is to ensure your contributions meet our guidelines. I've found some issues in your pull request that should be addressed (click on them for more details) 👇
|
Why
Part of ENG-2401.
How
Send uncaught exceptions to the bundler server if possible. Our dev server has its endpoint where we can send logs. So I've just used it.
Test Plan