Issue #7080 has been updated by Daniel Pittman.
Status changed from Ready For Testing to Code Insufficient
Target version set to Statler
This turns out to cause a regression in the indirector, leading to a failing
test and highlighting a problem with the code.
Specifically, once this branch is merged on top of 2.7.x the REST indirector
will eat the parameter `ip` in either GET or POST requests. This same test
passes on 2.7.x raw. The test is `spec/unit/indirector/rest_spec.rb` like
`231':
<pre>
describe "with a large body" do
it "should use the POST http method" do
params = {}
'aa'.upto('zz') do |s|
params[s] = 'foo'
end
@request = Puppet::Indirector::Request.new(:foo, :find, "foo bar",
params.merge(:environment => "myenv"))
@connection.expects(:post).with do |uri, body|
uri == "/myenv/foo/foo%20bar" and body.split("&").sort == params.map
{|key,value| "#{key}=#{value}"}.sort
end.returns(@response)
@searcher.find(@request)
end
end
</pre>
----------------------------------------
Feature #7080: Indirector requests should be serializable
https://projects.puppetlabs.com/issues/7080
Author: Luke Kanies
Status: Code Insufficient
Priority: Normal
Assignee: Luke Kanies
Category:
Target version: Statler
Affected Puppet version:
Keywords:
Branch: luke/tickets/next/7080-serializable_indirector_requests
This would be useful for interacting with message buses.
You could then pretty easily build an RPC system on the bus, sending indirector
requests over the wire and getting instances back.
--
You have received this notification because you have either subscribed to it,
or are involved in it.
To change your notification preferences, please click here:
http://projects.puppetlabs.com/my/account
--
You received this message because you are subscribed to the Google Groups
"Puppet Bugs" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/puppet-bugs?hl=en.
