-
Notifications
You must be signed in to change notification settings - Fork 8.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
HADOOP-18249. Fix getUri() in HttpRequest has been deprecated. #4335
Conversation
WebHdfsHandler.java req.getUri() replace uri(), req.getMethod() replace method() HostRestrictingAuthorizationFilterHandler.java req.getUri() replace uri() TestHostRestrictingAuthorizationFilterHandler.java remove throws Exception, channelResponse.getStatus() replace status().
🎊 +1 overall
This message was automatically generated. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks for the PR @slfan1989 , there were few more places where httprequest getUri() is used, can you modify them as well ?
Thanks @hemanthboyina , I'll try to fix the other locations. |
💔 -1 overall
This message was automatically generated. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
will wait for build result
🎊 +1 overall
This message was automatically generated. |
@hemanthboyina please help me to review the code again, thank you very much! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
@hemanthboyina Thank you very much for your help reviewing the code! |
…e#4335) * HADOOP-18249. Fix getUri() in HttpRequest has been deprecated. WebHdfsHandler.java req.getUri() replace uri(), req.getMethod() replace method() HostRestrictingAuthorizationFilterHandler.java req.getUri() replace uri() TestHostRestrictingAuthorizationFilterHandler.java remove throws Exception, channelResponse.getStatus() replace status(). * HADOOP-18249. Fix getUri() in HttpRequest has been deprecated. * HADOOP-18249. Fix Some CheckStyle. Co-authored-by: slfan1989 <louj1988@@>
Description of PR
JIRA: HADOOP-18249. Fix getUri() in HttpRequest has been deprecated.
When reading the code, I found that the method used has been deprecated due to the upgrade of the netty component. The main methods are as follows:
io.netty.handler.codec.http#HttpRequest
io.netty.handler.codec.http#DefaultHttpResponse