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

Set _APPSIGNAL_PROCESS_NAME config #32

Open
2 tasks
Tracked by #14
tombruijn opened this issue May 2, 2023 · 2 comments
Open
2 tasks
Tracked by #14

Set _APPSIGNAL_PROCESS_NAME config #32

tombruijn opened this issue May 2, 2023 · 2 comments
Labels

Comments

@tombruijn
Copy link
Member

To do

  • Figure out how to get the Python process name
  • Set it as _APPSIGNAL_PROCESS_NAME
@tombruijn
Copy link
Member Author

Looks like there's no $PROCESS_NAME like global or env var in Python from which we can fetch it. A lot of links point to https://pypi.org/project/psutil/, but for such a small thing I don't know if we should add a dependency with all that functionality right now.

In the end it fetches it from the proc dir for Linux, uses sysctl for macOS, and we don't support Windows. Not sure about FreeBSD.

I suggest we leave it for a little later. If it was small it was a nice addition, if it's a lot of work/dependency discussion let's postpone it.

@unflxw
Copy link
Contributor

unflxw commented May 9, 2023

I'd be okay with adding the dependency. 🤷

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants