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

Prevent 'Use of uninitialized value' warnings #42

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

willvincent
Copy link

When processes running within docker containers are running as users without an entry in the host machine's /etc/passwd file, many warnings will appear in the log. While they're just warnings and could be ignored, they do produce a great deal of excess noise that would be better to prevent.

One solution would be to create user entries in the passwd file, but that really shouldn't be necessary. Instead we can just cast the user value to an empty string to prevent the warnings entirely.

This resolves #24

…esses run as users NOT in host machine passwd file.
@wschaub
Copy link

wschaub commented Oct 1, 2018

Might be useful to just display the raw UID number instead of an empty string if it's not found in the password database. it's what ps(1) would do and what I would expect.

@willvincent
Copy link
Author

Whatever the case.. linode doesn't seem terribly interested in continuing to support this codebase, so I guess it's on those of us affected to implement the solution we prefer, but probably not expect that this PR will ever be merged.

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

Successfully merging this pull request may close these issues.

data getter for processes shouldn't assume that every uid has pwd entry
2 participants