Commit 98fec2d5 authored by Alison Monteiro's avatar Alison Monteiro Committed by Dan Abramov
Browse files

react-dev-utils/openBrowser now supports urls with 2+ params (#2076)

* react-dev-utils/openBrowser now supports urls with 2+ params

Fixed #2047

* react-dev-utils/openBrowser: encode url before open the browser
parent 0ffed85e
3 merge requests!12191Lim.Pisey.168:/Identified - We are currently investigating reports of missing build logs. The issue has been identified and a resolution is in progress. We will provide a further update when available.Mar 21, 09:02 UTC,!12853brikk,!5717Automatically extract project file structure from build bundle file
Showing with 1 addition and 1 deletion
+1 -1
...@@ -76,7 +76,7 @@ function startBrowserProcess(browser, url) { ...@@ -76,7 +76,7 @@ function startBrowserProcess(browser, url) {
// Try our best to reuse existing tab // Try our best to reuse existing tab
// on OS X Google Chrome with AppleScript // on OS X Google Chrome with AppleScript
execSync('ps cax | grep "Google Chrome"'); execSync('ps cax | grep "Google Chrome"');
execSync('osascript openChrome.applescript ' + url, { execSync('osascript openChrome.applescript "' + encodeURI(url) + '"', {
cwd: __dirname, cwd: __dirname,
stdio: 'ignore', stdio: 'ignore',
}); });
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment