search

mozilla / minion-zap-plugin

Minion ZAP Plugin
15 stars 8 forks source link

Spit out the progress as we go #23

Open yeukhon opened 11 years ago

yeukhon commented 11 years ago

It would be nice if zap or zap plugin can spit out progress as it goes... question is: can zap do that via api?

yeukhon commented 11 years ago

Simon said it is doable via API (thanks, I was going to test it later, but quick info hehe) I will get this on my priority list.

yeukhon commented 11 years ago

Okay. I have looked at this issue. This is a non-trivial fix for the following reason:

As we report we will write to the database so at the end of the scan we will get lots of duplicates. We need a mechanism to separate live view and saving. I need to give it a thought.

yeukhon commented 11 years ago

There is a trivial solution, which I consider a hack and inefficient:

Poll in the frontend (issue and issues page)

But on the other hand, it is quite difficult to do real incremental, progressive update (append / insert): what is new. This is a backend issue and should be dealt with socket We need to listen into a channel. That's the right solution.

Other fixes (new api + temporary session) seems like a hack, adds more code, but emulate the beahvior of listening to a channel.