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

Updater should be able to execute external (own) sript #202

Closed
hitam4450 opened this issue Dec 6, 2015 · 8 comments
Closed

Updater should be able to execute external (own) sript #202

hitam4450 opened this issue Dec 6, 2015 · 8 comments

Comments

@hitam4450
Copy link

Hello Developer community of OwnCloud,

How about the idea, to provide a possibility that the Updater provides a possibility to execute an external (own) script (e.g. php, shell, perl, etc) at a given point of the update process??

I am looking forward to discussing with you!

Have a Nice, relaxing Sunday!

regards, hitam4450

@mmattel
Copy link
Contributor

mmattel commented Jan 19, 2016

👍

I extend this to a pre and a post script possibility
Because we have to:

  • set proper permissions pre running the update
  • reset proper permissions when the update has finished

And thinking that we may have to use superuser rights, we may want to ask in a field in the admin page for the superusers password which we pass to the script...

@VicDeo
Copy link
Member

VicDeo commented Feb 8, 2016

It will be possible to wrap CLI application call inside own bash (or any other) script since 9.0

@mmattel
Copy link
Contributor

mmattel commented Mar 22, 2016

@VicDeo
For sure beeing to stupid, but would you mid to describe how / where this can be done. I have not found a description for. The updater app does not give me a place where to add external pre/post scripts...

@VicDeo
Copy link
Member

VicDeo commented Mar 22, 2016

via CLI:

preupgradescript.sh
sudo -u wwwrun php /path/to/owncloud/updater/application.php 
postupgradescript.sh

@mmattel
Copy link
Contributor

mmattel commented Mar 22, 2016

Thx, got that.
But is there also a way to define the scripts you want to pre/post run when you use the updater app via the browser?

@mmattel
Copy link
Contributor

mmattel commented Mar 22, 2016

Because when checking for a build, I am able to start the update process from there, and at this point I need to run the scripts manually from a shell = parallel administration...

@DeepDiver1975 DeepDiver1975 added this to the backlog milestone Apr 4, 2016
@DeepDiver1975
Copy link
Member

This needs really in depth thinking - we would need different handlings depending on the execution mode: web vs cli.

I'm not really convinced

@ghost
Copy link

ghost commented Oct 11, 2016

I’m closing this issue because it has been inactive for a few months. This probably means that it is not reproducible or it has been fixed in a newer version. If it’s an enhancement and hasn’t been taken on for so long, then it seems no one has the time to implement this.

Please reopen if you still encounter this issue with the latest stable version and then please use the issue template. You can also contribute directly by providing a patch – see the developer manual. :)

Thank you!

@ghost ghost closed this as completed Oct 11, 2016
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants