openedx-unsupported / edx-analytics-configuration

GNU Affero General Public License v3.0
8 stars 28 forks source link

Shorten ssh control path #60

Closed rao-abdul-mannan closed 5 years ago

rao-abdul-mannan commented 5 years ago

This PR fixes issue with update-users jenkins job on new jenkins server which failed due to long ssh control path. I have tested user-activity-edge workflow using this branch which finished successfully.

brianhw commented 5 years ago

Heh - the documentation I found says to switch it to the value we currently have if the socket pathnames get too long. : ) .

What does the %%C substitute? And is it going to be sufficiently different?

rao-abdul-mannan commented 5 years ago

@brianhw OpenSSH wiki says that starting version 6.7, the combination of %r@%h:%p and variations on it can be replaced with %C which by itself generates a SHA1 hash from the concatenation of %l%h%p%r. %C being hash can be shorter & it also obfuscates the connection information which would have been otherwise displayed in the socket's name.