wp-checksum icon indicating copy to clipboard operation
wp-checksum copied to clipboard

Define 'requests' in quota

Open ethanpil opened this issue 7 years ago • 3 comments

You state the free request limit is '30' per hour. Does that mean 30 files? 30 scans? 30 what?

ethanpil avatar May 12 '17 17:05 ethanpil

Hi,

You're right, this is not very clear in the current docs. Thanks for pointing that out.

A request is when the client asks for checksum info for one theme or plugin in your installation. So the number of plugins + the number of themes you have sums up to the number of requests needed for one complete sites.

Since the docs was last updated, the limit has been raised to 50/hour. That number will allow most (but not all) sites we've seen (so far) to have a free hourly scan.

Any feedback on this model is appreciated, we're eager to find a balance between being able to maintain an acceptable service level and providing a good service to the community.

eriktorsner avatar May 12 '17 18:05 eriktorsner

I think it makes sense for now.

Suggestions:

  1. Provide the limit information in the scan results:

API Limit: 50/hour | This Scan used XX credits | XX remaining this hour

  1. If we run out of API credit in the middle of a scan, it aborts. This wastes the entire scan time and credits used. If the scan aborts due to API limit, it should still provide results up to that point.

  2. If we do have more plugins/themes then remaining in our limit, it would be nice to be able to batch into X number of scans that can be performed over time.

For example wp checksum plugin --batch-start=1 --batch-end=15 would scan the first 15 plugins wp checksum plugin --batch-start=16 --batch-end=30 would scan the second 15 plugins

or maybe there is an even easier way...

ethanpil avatar May 18 '17 16:05 ethanpil

Hi,

I appreciate the improvement suggestions a lot, thanks for taking the time to think this through and comment. I think a combination of your suggestions will make a nice addition to the client.

I'll look at how to best implement and post updates on the progress in this issue.

Thanks,

/Erik

On Thu, May 18, 2017 at 6:08 PM, ethanpil [email protected] wrote:

I think it makes sense for now.

Suggestions:

  1. Provide the limit information in the scan results:

API Limit: 50/hour | This Scan used XX credits | XX remaining this hour

If we run out of API credit in the middle of a scan, it aborts. This wastes the entire scan time and credits used. If the scan aborts due to API limit, it should still provide results up to that point. 2.

If we do have more plugins/themes then remaining in our limit, it would be nice to be able to batch into X number of scans that can be performed over time.

For example wp checksum plugin --batch-start=1 --batch-end=15 would scan the first 15 plugins wp checksum plugin --batch-start=16 --batch-end=30 would scan the second 15 plugins

or maybe there is an even easier way...

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/eriktorsner/wp-checksum/issues/8#issuecomment-302454814, or mute the thread https://github.com/notifications/unsubscribe-auth/ACs2rwFplwWCfr3bJEqJJk0a0k28XyOTks5r7Gz8gaJpZM4NZlWl .

-- /Erik Torsner Tel: 0733-118503 Skype: eriktorsner

eriktorsner avatar May 21 '17 14:05 eriktorsner