status-im
/
metro
mirror of https://github.com/status-im/metro.git
2
0
Fork 0
Code Issues Projects Releases Wiki Activity

Reverted commit D3516741 

Summary:
Looks like spaces in function names can happen, but the lib we use for parsing stacktraces doesn't support that. As result, when error is thrown in global scope, new JSC puts "global code" as function name, our parser chokes on it and thinks "global code@http://...." is a file name and sends it to packager. The packager can't resolve that URL and fails the whole symbolication request.

Longer term fix here: https://github.com/errwischt/stacktrace-parser/pull/5

Reviewed By: astreet

Differential Revision: D3516741

fbshipit-source-id: 7f14b52a50a118dc95a3463aee842941e904e984
This commit is contained in:
Konstantin Raev 2016-07-05 11:36:31 -07:00 committed by Facebook Github Bot 6
parent 1dcf2e0c7f
commit e428436416
1 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
react-packager/src/Server/index.js vendored
View File

@ -499,10 +499,8 @@ class Server {
const sourceUrl = frame.file; const sourceUrl = frame.file;
// Skip `/debuggerWorker.js` which drives remote debugging because it // Skip `/debuggerWorker.js` which drives remote debugging because it
// does not need to symbolication. // does not need to symbolication.
// Skip anything except http(s), because there is no support for that yet
if (!urlIndexes.hasOwnProperty(sourceUrl) && if (!urlIndexes.hasOwnProperty(sourceUrl) &&
!sourceUrl.endsWith('/debuggerWorker.js') && !sourceUrl.endsWith('/debuggerWorker.js')) {
sourceUrl.startsWith('http')) {
urlIndexes[sourceUrl] = uniqueUrls.length; urlIndexes[sourceUrl] = uniqueUrls.length;
uniqueUrls.push(sourceUrl); uniqueUrls.push(sourceUrl);
} }