Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add the plugin to the safe.directory list #92

Merged
merged 1 commit into from
May 19, 2022
Merged

Conversation

josegonzalez
Copy link
Member

This allows dokku to trigger plugin:update as root while keeping plugins owned by the Dokku user.

See https://github.blog/2022-04-12-git-security-vulnerability-announced/ for more details on this change (which was backported in to Git 2.25.1 for Ubuntu).

Closes dokku/dokku#5186

This allows dokku to trigger plugin:update as root while keeping plugins owned by the Dokku user.

See https://github.blog/2022-04-12-git-security-vulnerability-announced/ for more details on this change (which was backported in to Git 2.25.1 for Ubuntu).

Closes dokku/dokku#5186
@josegonzalez josegonzalez merged commit ba2f9d3 into master May 19, 2022
@josegonzalez josegonzalez deleted the handle-safe-dirs branch May 19, 2022 04:05
josegonzalez added a commit that referenced this pull request May 19, 2022
### Fixed

- @josegonzalez Ignore plugins that do not have a .git directory when performing updates #90
- @josegonzalez Add the plugin to the safe.directory list #92

### Changed

- @josegonzalez Do not require PLUGIN_PATH to fetch version #91
- @josegonzalez Log when a plugin trigger is not executable #93
@josegonzalez josegonzalez mentioned this pull request May 19, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can't update plugins due to permissions issue
1 participant