devmadhuu commented on PR #3853: URL: https://github.com/apache/ozone/pull/3853#issuecomment-1281942823
> @devmadhuu this might not be in scope for this change, but we need a limit applied server side as well. The client should not be able to send an arbitrarily large count to the server. Thanks @kerneltime for reviewing the PR, Understood the point. I have added the config on server side as well and below will be the behavior: if client sends the config value for num of listing entries higher than server side config, then cap at server side val, If client sends less than what is configured at server side, then send same number of entries as what client sent. Request you to pls re-review -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
