The Puppet Labs Issue Tracker has Moved: https://tickets.puppetlabs.com

Bug #1417

ssh_autorized_key cannot parse lines which a command containing spaces

Added by Francois Deppierraz about 6 years ago. Updated almost 6 years ago.

Status:ClosedStart date:07/11/2008
Priority:NormalDue date:
Assignee:Francois Deppierraz% Done:

0%

Category:newfeature
Target version:0.25.0
Affected Puppet version:0.24.4 Branch:
Keywords:

We've Moved!

Ticket tracking is now hosted in JIRA: https://tickets.puppetlabs.com

This issue is currently not available for export. If you are experiencing the issue described below, please file a new ticket in JIRA. Once a new ticket has been created, please add a link to it that points back to this Redmine ticket.


Description

Parsing of the following line

command="rsync --server -vlogDtprCz . /",no-pty,no-port-forwarding,no-X11-forwarding ssh-dss AAAAB Backup

results in

Puppet::Error in 'Puppet::Type::Ssh_authorized_key::ProviderParsed should be able to parse each example'
Could not parse line "command=\"rsync --server -vlogDtprCz . /\",no-pty,no-port-forwarding,no-X11-forwarding ssh-dss AAAAB Backup" at /home/francois/dev/puppet/puppet/test/data/providers/ssh_authorized_key/parsed/long-command:2
/home/francois/dev/puppet/puppet/spec/../lib/puppet/util/fileparsing.rb:240:in `parse'
/home/francois/dev/puppet/puppet/spec/../lib/puppet/util/fileparsing.rb:233:in `collect'
/home/francois/dev/puppet/puppet/spec/../lib/puppet/util/fileparsing.rb:233:in `parse'
/home/francois/dev/puppet/puppet/spec/../lib/puppet/provider/parsedfile.rb:236:in `retrieve'
/home/francois/dev/puppet/puppet/spec/../lib/puppet/provider/parsedfile.rb:201:in `prefetch_target'
/home/francois/dev/puppet/puppet/spec/../lib/puppet/provider/parsedfile.rb:176:in `prefetch'
/home/francois/dev/puppet/puppet/spec/../lib/puppet/provider/parsedfile.rb:175:in `each'
/home/francois/dev/puppet/puppet/spec/../lib/puppet/provider/parsedfile.rb:175:in `prefetch'
/home/francois/dev/puppet/puppet/test/lib/puppettest/fileparsing.rb:7:in `fakedataparse'
/home/francois/dev/puppet/puppet/test/lib/puppettest/fileparsing.rb:4:in `each'
/home/francois/dev/puppet/puppet/test/lib/puppettest/fileparsing.rb:4:in `fakedataparse'
./parsed.rb:47:
./parsed.rb:45:in `each'
./parsed.rb:45:
/home/francois/dev/puppet/puppet/spec/monkey_patches/add_confine_and_runnable_to_rspec_dsl.rb:19:in `run'
/home/francois/dev/puppet/puppet/spec/monkey_patches/add_confine_and_runnable_to_rspec_dsl.rb:17:in `each'
/home/francois/dev/puppet/puppet/spec/monkey_patches/add_confine_and_runnable_to_rspec_dsl.rb:17:in `run'
./parsed.rb:11:

fix-1047.patch Magnifier - allow whitespace in options part (575 Bytes) Brice Figureau, 07/30/2008 11:27 am

History

#1 Updated by James Turnbull about 6 years ago

  • Category set to newfeature
  • Status changed from Accepted to Needs More Information
  • 3 changed from Unknown to Trivial

Any update on this?

#2 Updated by James Turnbull about 6 years ago

  • Target version changed from 0.24.5 to 0.25.0

#3 Updated by Brice Figureau almost 6 years ago

Here is a small patch that allows to use whitespace in the options part of the ssh_authorized_keys. So far, it seemed to work fine on my systems.

#4 Updated by James Turnbull almost 6 years ago

  • Status changed from Needs More Information to Closed

Pushed in commit:0bbac8dda9342b28782ae1218cdb891f5a5bfa54 in branch 0.24.x

Also available in: Atom PDF