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

Passing arguments with cron #106

Open
TGates71 opened this issue Jan 30, 2015 · 5 comments
Open

Passing arguments with cron #106

TGates71 opened this issue Jan 30, 2015 · 5 comments
Assignees
Milestone

Comments

@TGates71
Copy link
Member

Users can not pass arguments for cron jobs.

@MBlagui
Copy link
Contributor

MBlagui commented Feb 2, 2015

We should offer 2 ways CLI and HTTP.

HTTP could use WGET + http URL that would be easy to implet and no sandboxing issues here.

Also if we add back PERL or third party languages, how we deal with them? The current cron job way is only tailored for PHP. So we should think about the future.

@Edabio
Copy link

Edabio commented Apr 8, 2015

preciso mudar minhas tarefas cron para o plugin Snap de AutoPost para não acontecer de publicações repetidas. O que fazer neste caso!

@5050
Copy link
Contributor

5050 commented Apr 8, 2015

@Edabio :

  1. we do not read spanish. Sorry. Please try to write in EN.
  2. the bug tracker is not a place for support. please use the forum
  3. your last question (about adding Subdomain) have nothing to do with the subject : Cron task. I'll delete it. Please post it in the forum : http://forums.sentora.org

@Caffe1neAdd1ct
Copy link
Contributor

We need to be very careful here, allowing args would open up the possibility for command injections via cron.

@Caffe1neAdd1ct Caffe1neAdd1ct added this to the Backlog milestone Sep 28, 2015
@Caffe1neAdd1ct Caffe1neAdd1ct self-assigned this Sep 28, 2015
@MBlagui
Copy link
Contributor

MBlagui commented Sep 30, 2015

Arguments should be only using HTTP ! not CLI le'ts be clear over that.

How you we will get command injections if we run the request over http? @Caffe1neAdd1ct

M B

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