Make rate limits configureable

[kannapoix]

Closes #49
Make rate limits period and threshold configurable by adding 'rate_limit_count' and 'rate_limit_period' to configuration option.

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Project coverage is 0.00%. Comparing base (259de8b) to head (8d34e5b).
Report is 31 commits behind head on master.

Files with missing lines	Patch %	Lines
src/main.rs	0.00%	2 Missing ⚠️

Additional details and impacted files

@@          Coverage Diff           @@
##           master    #153   +/-   ##
======================================
  Coverage    0.00%   0.00%           
======================================
  Files           1       1           
  Lines         105     128   +23     
======================================
- Misses        105     128   +23     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[orbitalturtle]

Thanks for this PR @kannapoix :) I'll finally take a look on Monday.

[dunxen]

This dropped off my radar recently. ごめんなさい 🙇

Feel free to squash the last commit for specifying the secs suffix.

Otherwise this LGTM!

[kannapoix]

@dunxen No problems. 確認ありがとうございます ✨
Commits squashed.

When is the right time to squash commits?
I thought it was bad to squash commits during a review, because changing the commit number makes comments unassociated with a commit. So I always squash them after a reviewer asks me to(just like you did).
Is this OK, or should I organize commits when I feel it is necessary?

[dunxen]

When is the right time to squash commits? I thought it was bad to squash commits during a review, because changing the commit number makes comments unassociated with a commit. So I always squash them after a reviewer asks me to(just like you did). Is this OK, or should I organize commits when I feel it is necessary?

You are 100% correct! You did the right thing to leave the fixup commit for review.

[dunxen]

I'll wait a little while for @orbitalturtle to have a final look, but this does LGTM so I think it can be merged some time today, otherwise :)