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

Allow passing hardcoded uid to AuthExternal #188

Closed
kiwiz opened this issue Dec 23, 2022 · 1 comment · Fixed by #189
Closed

Allow passing hardcoded uid to AuthExternal #188

kiwiz opened this issue Dec 23, 2022 · 1 comment · Fixed by #189
Labels
enhancement New feature or request

Comments

@kiwiz
Copy link
Contributor

kiwiz commented Dec 23, 2022

Is your feature request related to a problem? Please describe.

I'm trying to connect to the system DBus within a Docker container that is utilizing a user namespace. This fails, as the os.uid() call here returns the remapped UID (which is entirely expected).

Describe the solution you'd like

For my use case, I would like to pass a hardcoded UID to AuthExternal, perhaps via the constructor or an env variable. If this sounds like a reasonable change, I can submit a PR.

Additional context

I arrived here via the following path:

Let me know if this feature request would be better suited for one of those repos.

@kiwiz kiwiz added the enhancement New feature or request label Dec 23, 2022
@bdraco
Copy link
Member

bdraco commented Dec 23, 2022

We can accept a PR for this via the constructor as an optional keyword arg as long as it has test coverage 👍

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

Successfully merging a pull request may close this issue.

2 participants