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

Register valid root for all installed apps in console #19022

Merged
merged 1 commit into from
Sep 16, 2015

Conversation

RobinMcCorkell
Copy link
Member

Might be unnecessary, but I think there are some cases where we want to be able to run app commands after the app is disabled, to perform last-chance maintenance tasks. Negligible security impact, since this only gets triggered from the CLI.

cc @nickvergessen @DeepDiver1975 @MorrisJobke

Fixes #19019

We need to use commands from apps that aren't enabled (so don't get loaded).
@scrutinizer-notifier
Copy link

A new inspection was created.

@DeepDiver1975
Copy link
Member

I guess this is no longer needed because of https://github.com/owncloud/core/pull/19039/files

@nickvergessen @Xenopathic please retest #19019 with latest master - THX

@nickvergessen
Copy link
Contributor

#19039 didn't help, but with this PR it works.

#19039 only helps when apps depend on each others, but they are still not loaded in the console path

@nickvergessen
Copy link
Contributor

👍 fixes ./occ maintenance:mode --off

@nickvergessen
Copy link
Contributor

@Xenopathic please rebase for CI coverage

@DeepDiver1975
Copy link
Member

@Xenopathic please rebase for CI coverage

no needed

@DeepDiver1975
Copy link
Member

👍

DeepDiver1975 added a commit that referenced this pull request Sep 16, 2015
Register valid root for all installed apps in console
@DeepDiver1975 DeepDiver1975 merged commit b0414f6 into master Sep 16, 2015
@DeepDiver1975 DeepDiver1975 deleted the console-loader-validroot branch September 16, 2015 13:17
@lock lock bot locked as resolved and limited conversation to collaborators Aug 10, 2019
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.

Installing via console fails
4 participants