Skip to content

Throwing error message incase of lengthy payload#19

Merged
migueldemoura merged 1 commit intocloudflare:masterfrom
shreyavshetty:add_truncation
Feb 15, 2022
Merged

Throwing error message incase of lengthy payload#19
migueldemoura merged 1 commit intocloudflare:masterfrom
shreyavshetty:add_truncation

Conversation

@shreyavshetty
Copy link
Contributor

@shreyavshetty shreyavshetty commented Feb 10, 2022

Payload match exceeding a certain limit will be truncated and the changes are made to throw an error message for the same.

@migueldemoura
Copy link
Contributor

Builds are failing, can you run cargo fmt?

@migueldemoura
Copy link
Contributor

Can you sign your commits, @shreyavshetty (and merge the 1st and 2nd ones as the truncation bit, and rename the 3rd one to "Update dependencies" or something similar)?

@shreyavshetty shreyavshetty changed the title Throwing error message incase of lengthy payload Adding dependencies Feb 11, 2022
@shreyavshetty shreyavshetty changed the title Adding dependencies Throwing error message incase of lengthy payload Feb 11, 2022
@shreyavshetty
Copy link
Contributor Author

Can you sign your commits, @shreyavshetty (and merge the 1st and 2nd ones as the truncation bit, and rename the 3rd one to "Update dependencies" or something similar)?

Squashed to one, and signed

@migueldemoura migueldemoura merged commit ce8c002 into cloudflare:master Feb 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants