Closed Bug 1551689 Opened 6 months ago Closed 6 months ago

Print listener address with a new-line

Categories

(Remote Protocol :: Agent, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: ochameau, Unassigned)

References

Details

Attachments

(1 file)

I missed during the review of bug 1551188 the lack of \n on the following line:
https://searchfox.org/mozilla-central/rev/116bd975c30746ddefc3d20e6947d1871469354f/remote/RemoteAgent.jsm#103
dump(DevTools listening on ${mainTarget.wsDebuggerURL});

It breaks puppeteer as this line ends up being mixed up with the next line logged by Firefox and ends up being merged with the debugger URL in puppeteer code.

Pushed by atolfsen@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/aae069bec8e5
Print listener address with a new-line. r=remote-protocol-reviewers,ato
Status: NEW → RESOLVED
Closed: 6 months ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.