Skip to content

Update to ratelimit 0.0.6#148

Merged
ehsandeep merged 1 commit intoprojectdiscovery:devfrom
mikelolasagasti:ratelimit-0.0.6
Feb 25, 2023
Merged

Update to ratelimit 0.0.6#148
ehsandeep merged 1 commit intoprojectdiscovery:devfrom
mikelolasagasti:ratelimit-0.0.6

Conversation

@mikelolasagasti
Copy link
Copy Markdown
Contributor

@mikelolasagasti mikelolasagasti commented Feb 24, 2023

Proposed changes

Newer ratelimit replaces signed int64 with uint since counters are always positive down to zero as per projectdiscovery/ratelimit#6 so this PR changes the call to ratelimit and updates required version.

Checklist

  • Pull request is created against the dev branch
  • All checks passed (lint, unit/integration/regression tests etc.) with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Feb 24, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@ehsandeep ehsandeep merged commit 350f928 into projectdiscovery:dev Feb 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ehsandeep ehsandeep ehsandeep approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mikelolasagasti @ehsandeep