alluxio icon indicating copy to clipboard operation
alluxio copied to clipboard

Extract Authentication and common logging into the specific filters

Open Jackson-Wang-7 opened this issue 2 years ago • 1 comments

What changes are proposed in this pull request?

Extract Authentication and common logging into the specific filters

Why are the changes needed?

I added two filters, one is for authentication and another is for logging. we have several implementations to log the request parameter, and one of them is in CompleteMultipartUploadHandler. it will confuse other developers. And we need a common filter to perform authentication. it will be helpful for us to further develop.

Jackson-Wang-7 avatar Sep 14 '22 03:09 Jackson-Wang-7

is the new code tested?

@ZhuTopher Please take a look to see if it is proper.

yyongycy avatar Sep 15 '22 01:09 yyongycy

alluxio-bot, merge this please

yyongycy avatar Sep 23 '22 03:09 yyongycy