Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix SSH passwordless check for OpenSSH #2280

Merged
merged 1 commit into from Aug 11, 2012
Merged

fix SSH passwordless check for OpenSSH #2280

merged 1 commit into from Aug 11, 2012

Conversation

ikn
Copy link
Contributor

@ikn ikn commented Aug 10, 2012

pexpect takes a regular expression, so we want '[Pp]assword:', not
'[Ppassword]:'.

pexpect takes a regular expression, so we want '[Pp]assword:', not
'[Ppassword]:'.
@takluyver
Copy link
Member

Test results for commit f9a87d2 merged into master (2092555)
Platform: linux2

  • python2.7: OK (libraries not available: oct2py pymongo wx wx.aui)
  • python3.2: OK (libraries not available: oct2py pymongo wx wx.aui)

Not available for testing: python2.6

@takluyver
Copy link
Member

Oops, meant to do a --all test run, as this affects IPython.parallel.

@takluyver
Copy link
Member

Test results for commit f9a87d2 merged into master (2092555)
Platform: linux2

  • python2.7: OK (libraries not available: oct2py pymongo wx wx.aui)
  • python3.2: OK (libraries not available: oct2py pymongo wx wx.aui)

Extra args: ['--all']
Not available for testing: python2.6

@bfroehle
Copy link
Contributor

Looks good to me. Works fine manually testing on a few different sites.

In [2]: _try_passwordless_openssh('root@localhost', None)
Out[2]: False

In [3]: _try_passwordless_openssh('localhost', None)
Out[3]: True

@ellisonbg
Copy link
Member

This one looks pretty straightforward, can it be merged? @minrk do you want to have a look?

@minrk
Copy link
Member

minrk commented Aug 11, 2012

Yes, this is right. Merging away, thanks!

minrk added a commit that referenced this pull request Aug 11, 2012
fix SSH passwordless check for OpenSSH

typo in pexpect pattern

closes #2266
@minrk minrk merged commit 31afb62 into ipython:master Aug 11, 2012
minrk added a commit to minrk/pyzmq that referenced this pull request Aug 19, 2012
minrk added a commit that referenced this pull request Aug 22, 2012
pexpect takes a regular expression, so we want '[Pp]assword:', not
'[Ppassword]:'.
minrk added a commit that referenced this pull request Sep 1, 2012
pexpect takes a regular expression, so we want '[Pp]assword:', not
'[Ppassword]:'.
yarikoptic added a commit to yarikoptic/ipython that referenced this pull request May 2, 2014
* commit 'rel-0.13-33-gcfc5692': (33 commits)
  Backport PR ipython#2347: adjust division error message checking to account for Python 3
  Backport PR ipython#2305: RemoteError._render_traceback_ calls self.render_traceback
  Backport PR ipython#2280: fix SSH passwordless check for OpenSSH
  Backport PR ipython#2270: SSHLauncher tweaks
  Backport PR ipython#2261: Fix: longest_substr([]) -> ''
  Backport PR ipython#2250: fix html in notebook example
  Backport PR ipython#2235: remove spurious print statement from setupbase.py
  fixup
  Backport PR ipython#2223: Custom tracebacks
  Backport PR ipython#2214: use KernelApp.exec_lines/files in IPEngineApp
  Backport PR ipython#2212: catch errors in markdown javascript
  Backport PR ipython#2194: clean nan/inf in json_clean
  Backport PR ipython#2177: remove numpy install from travis/tox scripts
  Backport PR ipython#2169: ipdb: pdef, pdoc, pinfo magics all broken
  Backport PR ipython#2186: removed references to h5py dependence in octave magic documentation
  Backport PR ipython#2185: added test for %store, fixed storemagic
  Backport PR ipython#2170: Fix tab completion with IPython.embed_kernel().
  Backport PR ipython#2163: fix 'remote_profie_dir' typo in SSH launchers
  Backport PR ipython#2117: use explicit url in notebook example
  Backport PR ipython#2126: ipcluster broken with any batch (PBS/LSF/SGE)
  ...
mattvonrocketstein pushed a commit to mattvonrocketstein/ipython that referenced this pull request Nov 3, 2014
fix SSH passwordless check for OpenSSH

typo in pexpect pattern

closes ipython#2266
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants