lookupAddress(node) has non-standard behavior due to not lowercasing address

[marekkirejczyk]

As specified here reverse ENS address should be lowered case, while lookupAddress method is using utils.getAddress, which will return address mixed case.

I suspect in file provider.js line 968 instead of

address = utils.getAddress(address);

should be

address = utils.getAddress(address).toLowerCase();

[ricmoo]

Are you having an issue with this not working? Can you provide an example to reproduce the issue?

The line immediately afterwards calls namehash, which performs toLowerCase() on the provided name. The checksum address is required later in the function when comparing to the result from the ENS contracts.

Thanks! :)

[ricmoo]

Closing this, but if you have an example that fails to resolve, please feel re-open this issue.

Thanks! :)

[marekkirejczyk]

I have checked again. The problem was that reverse lookup result didn't point to the domain pointing to the same address.
So, all is good. Thanks!