[Dubbo-RpcContext] Simplify check consumer side in RpcContext

[imsunv]

What is the purpose of the change

url.getPort() != address.getPort() ||
                !NetUtils.filterLocalHost(url.getIp()).equals(NetUtils.filterLocalHost(host));

There is no need to compare address to confirm whether they are consumer or not in RpcContext.
the remote address comes from url and compare with url.

Brief changelog

RpcContext

[codecov-io]

Codecov Report

Merging #1444 into master will increase coverage by 0.01%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master   #1444      +/-   ##
=========================================
+ Coverage   31.59%   31.6%   +0.01%     
=========================================
  Files         682     682              
  Lines       33035   33015      -20     
  Branches     6592    6583       -9     
=========================================
- Hits        10436   10435       -1     
+ Misses      20748   20731      -17     
+ Partials     1851    1849       -2

Impacted Files	Coverage Δ
...rc/main/java/com/alibaba/dubbo/rpc/RpcContext.java	29.78% <0%> (+3.7%)	⬆️
...ubbo/rpc/protocol/dubbo/ChannelWrappedInvoker.java	41.66% <0%> (-8.34%)	⬇️
...bo/remoting/transport/netty/NettyCodecAdapter.java	55.22% <0%> (-1.5%)	⬇️
...a/dubbo/remoting/transport/netty/NettyChannel.java	66.25% <0%> (+5%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1f22f56...5191ffe. Read the comment docs.

[cvictory]

where is the usage of isProviderSide and isConsumerSide?
I cannot judge whether it is right.

[cvictory]

Usually the commit is right.

Is there some special situation happened？

[imsunv]

the isProviderSide and isConsumerSide seem not use in dubbo framework. just a api for user.
and the origin implementation is almost right.
but when we set DUBBO_IP_TO_REGISTRY use hostname not ip , the check is wrong.
and maybe lookup dns to resolve hostname in implementation.

[imsunv]

It's depends on the side key in new implementation. we can also see other place in dubbo code to use side key to check is consumer side or not.

[chickenlj]

I think SIDE_KEY is enough to determine provider or consumer.