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

Exporting feedback crashed api and it didn't come back #3107

Closed
garethbowen opened this issue Feb 8, 2017 · 4 comments
Closed

Exporting feedback crashed api and it didn't come back #3107

garethbowen opened this issue Feb 8, 2017 · 4 comments
Assignees
Labels
Type: Bug Fix something that isn't working as intended

Comments

@garethbowen
Copy link
Member

Exporting feedback on lg.app crashed api with this error. The error occurred about 5 minutes after the initial feedback request and even after the initial request had 502ed.

#
# Fatal error in ../deps/v8/src/handles.h, line 48
# CHECK(location_ != NULL) failed
#
 
2017-02-08T03:28:14.491Z - error: 
==== C stack trace ===============================

 1:  
2017-02-08T03:28:14.492Z - error: V8_Fatal
 2:  
2017-02-08T03:28:14.492Z - error: v8::String::NewFromUtf8(v8::Isolate*, char const*, v8::String::NewStringType, int)
 3: node::StringBytes::Encode(v8::Isolate*, char const*, unsigned long, node::encoding)
 4: node::Buffer::Utf8Slice(v8::FunctionCallbackInfo<v8::Value> const&)
 5: v8::internal::FunctionCallbackArguments::Call(void (*)(v8::FunctionCallbackInfo<v8::Value> const&))
 6: ??
 7: ??

This could be a node bug, or something else.

Also concerning was that the process didn't exit, so gardener didn't restart it. It had to be manually killed.

@ghost
Copy link

ghost commented Feb 28, 2017

This is a V8 bug that ultimately resulted in nodejs/node#649. It was fixed late in the 0.12.x series. Medic OS is now on Node v6.95 (LTS) and the bug should be long gone post-upgrade.

@ngaruko
Copy link
Contributor

ngaruko commented Apr 4, 2017

Are you able to confirm that this was fixed @garethbowen ? Is there a need to AT this?

@garethbowen
Copy link
Member Author

@ngaruko Yes this needs AT.

@garethbowen garethbowen assigned ngaruko and unassigned ghost Apr 4, 2017
@sglangevin
Copy link

This needs to be AT-ed on a server with updated Medic OS.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Fix something that isn't working as intended
Projects
None yet
Development

No branches or pull requests

3 participants