process.title don't change process name #28945

CubixSystem · 2019-08-02T18:06:21Z

Version: v10.16.1
Platform: 18.7.0 Darwin Kernel Version 18.7.0: Thu Jun 20 18:42:21 PDT 2019; root:xnu-4903.270.47~4/RELEASE_X86_64 x86_64
Subsystem: process

Hello

In v10.16.1

process.title = "test";

stops changing process name

In v10.15.3
All works as expected

The text was updated successfully, but these errors were encountered:

addaleax · 2019-08-02T18:08:06Z

/cc @nodejs/platform-macos since on Linux this seems to work

CubixSystem · 2019-08-02T18:19:24Z

I use nvm, my steps to reproduce:

nvm use v10.15.3 && /
node -e "process.title = 'process.title test'; process.stdin.resume();"

nvm use v10.16.1 && /
node -e "process.title = 'process.title test'; process.stdin.resume();"

jasnell · 2019-08-02T21:19:35Z

Can I ask you to check to see if the current Node.js 12.x release works? Like to see if this regression is specific to 10.x or not.

Trott · 2019-08-02T21:28:25Z

Can I ask you to check to see if the current Node.js 12.x release works? Like to see if this regression is specific to 10.x or not.

I can reproduce the issue in 12.0.0 and 12.7.0. Like original reporter, I'm seeing the problem in 10.16.1 but not in 10.15.3. I also see the problem in 10.16.0.

Trott · 2019-08-02T21:30:26Z

8.16.0 does not have the bug.

So, to summarize for supported release lines on macOS:

All 12.x are affected.
10.16.0 and newer are affected. 10.15.3 and earlier are not affected.
All 8.x are not affected.

richardlau · 2019-08-02T21:41:19Z

Sounds like it could be libuv related based on the Node.js versions affected. Node.js 10.16.0 bumped libuv from 1.23.2->1.28.0.

jasnell · 2019-08-02T21:58:49Z

Windows also appears to work as expected

richardlau · 2019-08-03T06:08:55Z

Maybe libuv/libuv@00c6b16#diff-60fd60fae228080a06862eae92a6e25 which came with libuv 1.24.0?

Trott · 2019-08-03T06:18:05Z

git bisect confirms that this bug was introduced with the upgrade to libuv 1.24.0 in 9cef7b8. I suppose the thing to do is open an issue in the libuv repository. @nodejs/libuv

This reverts commit 00c6b16. It's been reported (and I can confirm) that this change breaks `process.title = 'foo'` in Node.js. Since libuv just calls out to Core Services and Application Services, and since those frameworks are really just black boxes that you can't look inside, it's impossible to debug what exactly goes wrong. Revert it is then. Fixes: nodejs/node#28945

bnoordhuis · 2019-08-03T10:20:09Z

I can confirm that libuv/libuv@00c6b16 is responsible (but don't ask me why.) I've opened a revert in libuv/libuv#2405.

This reverts commit 00c6b16. It's been reported (and I can confirm) that this change breaks `process.title = 'foo'` in Node.js. Since libuv just calls out to Core Services and Application Services, and since those frameworks are really just black boxes that you can't look inside, it's impossible to debug what exactly goes wrong. Revert it is then. Fixes: nodejs/node#28945 PR-URL: libuv#2405 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Saúl Ibarra Corretgé <saghul@gmail.com>

addaleax · 2019-08-03T20:06:44Z

Re-opening since this isn’t fixed in Node.js yet.

ronkorving · 2019-08-04T01:02:12Z

And I think we'll want a regression test for this.

richardlau · 2019-08-04T01:18:57Z

There’s test-setproctitle.js. Perhaps someone on macOS can figure out why it isn’t currently failing?

cjihrig · 2019-08-04T04:09:32Z

Perhaps someone on macOS can figure out why it isn’t currently failing?

It looks like ps and other commands use a different data source than the Activity Monitor. I don't know much beyond that, but the commands in test-setproctitle.js do return the correct title for me.

Notable changes: - UV_FS_O_FILEMAP has been added for faster access to memory mapped files on Windows. - uv_fs_mkdir() now returns UV_EINVAL for invalid filenames on Windows. It previously returned UV_ENOENT. - The uv_fs_statfs() API has been added. - The uv_os_environ() and uv_os_free_environ() APIs have been added. Fixes: nodejs#28599 Fixes: nodejs#28945 Fixes: nodejs#29008 PR-URL: nodejs#29070 Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Jiawen Geng <technicalcute@gmail.com>

Notable changes: - UV_FS_O_FILEMAP has been added for faster access to memory mapped files on Windows. - uv_fs_mkdir() now returns UV_EINVAL for invalid filenames on Windows. It previously returned UV_ENOENT. - The uv_fs_statfs() API has been added. - The uv_os_environ() and uv_os_free_environ() APIs have been added. Fixes: #28599 Fixes: #28945 Fixes: #29008 PR-URL: #29070 Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Jiawen Geng <technicalcute@gmail.com>

Notable changes: - UV_FS_O_FILEMAP has been added for faster access to memory mapped files on Windows. - uv_fs_mkdir() now returns UV_EINVAL for invalid filenames on Windows. It previously returned UV_ENOENT. - The uv_fs_statfs() API has been added. - The uv_os_environ() and uv_os_free_environ() APIs have been added. Fixes: nodejs#28599 Fixes: nodejs#28945 Fixes: nodejs#29008 PR-URL: nodejs#29070 Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Jiawen Geng <technicalcute@gmail.com>

Notable changes: - UV_FS_O_FILEMAP has been added for faster access to memory mapped files on Windows. - uv_fs_mkdir() now returns UV_EINVAL for invalid filenames on Windows. It previously returned UV_ENOENT. - The uv_fs_statfs() API has been added. - The uv_os_environ() and uv_os_free_environ() APIs have been added. Fixes: #28599 Fixes: #28945 Fixes: #29008 PR-URL: #29070 Backport-PR-URL: #31969 Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Jiawen Geng <technicalcute@gmail.com>

coreybutler · 2020-07-09T04:00:15Z

This appears to be an issue again in 14.x.x. Works in 13.x.x.

cc @cjihrig @addaleax @bnoordhuis @Trott

bnoordhuis · 2020-07-09T09:01:00Z

@coreybutler Can you file a new issue and link back to this one?

coreybutler · 2020-07-09T16:27:58Z

@bnoordhuis sure, ^^

Many users assume the act of assigning a value to `process.title` will update the name of their application in apps like macOS Activity Monitor or Windows Services Manager. This has worked in the past, but fails in some versions of Node.js. Ultimately developers are left confused, especially when it works in one version of Node.js and not another. Given the recurring nature and complexity of the underlying problem, it does not seem like a resolvable problem. This note clarifies the source of the problem, sets developer expectations. Refs: #28945 PR-URL: #34557 Fixes: #34280 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com>

addaleax added the macos Issues and PRs related to the macOS platform / OSX. label Aug 2, 2019

Trott added confirmed-bug Issues with confirmed bugs. process Issues and PRs related to the process subsystem. labels Aug 2, 2019

bnoordhuis mentioned this issue Aug 3, 2019

Revert "darwin: speed up uv_set_process_title()" libuv/libuv#2405

Merged

bnoordhuis added the libuv Issues and PRs related to the libuv dependency or the uv binding. label Aug 3, 2019

bnoordhuis closed this as completed in libuv/libuv#2405 Aug 3, 2019

addaleax reopened this Aug 3, 2019

cjihrig mentioned this issue Aug 9, 2019

deps: upgrade to libuv 1.31.0 #29070

Merged

2 tasks

cjihrig closed this as completed in #29070 Aug 11, 2019

bnoordhuis mentioned this issue Sep 16, 2019

ci,macos: flaky process_title_threadsafe test libuv/libuv#2479

Closed

coreybutler mentioned this issue Jul 9, 2020

Example code not working on latest macOS coreybutler/node-mac#31

Open

coreybutler mentioned this issue Jul 9, 2020

process.title value is not respected on macOS (14.x.x+) #34280

Closed

coreybutler mentioned this issue Jul 29, 2020

doc: clarify process.title inconsistencies #34557

Closed

2 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

process.title don't change process name #28945

process.title don't change process name #28945

CubixSystem commented Aug 2, 2019

addaleax commented Aug 2, 2019

CubixSystem commented Aug 2, 2019

jasnell commented Aug 2, 2019

Trott commented Aug 2, 2019

Trott commented Aug 2, 2019

richardlau commented Aug 2, 2019

jasnell commented Aug 2, 2019

richardlau commented Aug 3, 2019

Trott commented Aug 3, 2019

bnoordhuis commented Aug 3, 2019

addaleax commented Aug 3, 2019

ronkorving commented Aug 4, 2019

richardlau commented Aug 4, 2019

cjihrig commented Aug 4, 2019

coreybutler commented Jul 9, 2020

bnoordhuis commented Jul 9, 2020

coreybutler commented Jul 9, 2020

process.title don't change process name #28945

process.title don't change process name #28945

Comments

CubixSystem commented Aug 2, 2019

addaleax commented Aug 2, 2019

CubixSystem commented Aug 2, 2019

jasnell commented Aug 2, 2019

Trott commented Aug 2, 2019

Trott commented Aug 2, 2019

richardlau commented Aug 2, 2019

jasnell commented Aug 2, 2019

richardlau commented Aug 3, 2019

Trott commented Aug 3, 2019

bnoordhuis commented Aug 3, 2019

addaleax commented Aug 3, 2019

ronkorving commented Aug 4, 2019

richardlau commented Aug 4, 2019

cjihrig commented Aug 4, 2019

coreybutler commented Jul 9, 2020

bnoordhuis commented Jul 9, 2020

coreybutler commented Jul 9, 2020