Mailbox address alias is not applied to refundRecipient

[antonper]

Hello,

The requestL2Transaction function applies the applyL1ToL2Alias to the msg.sender if the sender is smart contract.
However, in _requestL2Transaction the params.refundRecipient is assigned as:

 params.refundRecipient = _refundRecipient == address(0) ? msg.sender : _refundRecipient;

I think 2 things are not clear in this context and can be potential bugs:

1. The submitted refundRecipient address needs to be already aliased address, which is not clear from the docs.
2. The address(0) as refundRecipient will set wrong address if sender is a contract.

Please let me know if those concerns are valid.
Thank you

[ghost]

sorry but if you could nudgebme in the right direction ill try to do my best.. i have tons of tinkering files on the test and all kinds of adresses. i was just getting those gears going... hope i didnt piss off too many