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

NAS-133007 / 24.10.1 / Pass app for update method for ConfigService (by anodos325) #15175

Merged
merged 1 commit into from
Dec 10, 2024

Conversation

bugclerk
Copy link
Contributor

@bugclerk bugclerk commented Dec 10, 2024

When we have pair of update/do_update methods for a ConfigService we need to pass the authenticated credentials between them so that the latter method is called with appropriate privileges. If this doesn't happen, then in addition to the obvious problem of getting a privileged credential passed to the do_update method we also lose credential information for a job log if the method has a job decorator.

Original PR: #15173
Jira URL: https://ixsystems.atlassian.net/browse/NAS-133007

When we have pair of update/do_update methods for a ConfigService
we need to pass the authenticated credentials between them so that
the latter method is called with appropriate privileges. If this
doesn't happen, then in addition to the obvious problem of getting
a privileged credential passed to the do_update method we also
lose credential information for a job log if the method has a job
decorator.

(cherry picked from commit b14e7d9)
@anodos325 anodos325 marked this pull request as ready for review December 10, 2024 21:30
@anodos325 anodos325 merged commit b3b28d8 into release/24.10.1 Dec 10, 2024
1 check passed
@anodos325 anodos325 deleted the NAS-133007-24.10.1 branch December 10, 2024 21:31
@bugclerk
Copy link
Contributor Author

This PR has been merged and conversations have been locked.
If you would like to discuss more about this issue please use our forums or raise a Jira ticket.

@truenas truenas locked as resolved and limited conversation to collaborators Dec 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants