Smack icon indicating copy to clipboard operation
Smack copied to clipboard

Use XMPP connection as local socks5 address

Open MF1-MS opened this issue 3 years ago • 2 comments

The default local address is often just "the first address found in the list of addresses read from the OS" and this might mean an internal IP address that cannot reach external servers. So wherever possible use the same IP address being used to connect to the XMPP server because this local address has a better chance of being suitable.

This MR adds the above behaviour, and two UTs to test that we use the local XMPP connection IP when connected, and the previous behaviour when not.

MF1-MS avatar Aug 15 '22 15:08 MF1-MS

The default local address is often just "the first address found in the list of addresses read from the OS" and this might mean an internal IP address that cannot reach external servers. So wherever possible use the same IP address being used to connect to the XMPP server because this local address has a better chance of being suitable.

This MR adds the above behaviour, and two UTs to test that we use the local XMPP connection IP when connected, and the previous behaviour when not.

That is exactly what belongs in the commit message. Sadly the commit message right now does contain stuff that is just not true. ;)

Flowdalic avatar Aug 15 '22 18:08 Flowdalic

Commit message has been amended. :)

MF1-MS avatar Aug 17 '22 11:08 MF1-MS

OK done, should be good to merge now! Sorry I took so long to come make that whitespace change, coming back to assess your markups has been on the todo list a while! 😅

I just need to rebase this again quick...

MF1-MS avatar Apr 21 '23 15:04 MF1-MS

This is good to go.

MF1-MS avatar Apr 24 '23 10:04 MF1-MS