Issue #7474 has been updated by Mo Morsi.
Thank you for the feedback, updated patch sent to list. http://groups.google.com/group/puppet-dev/msg/2cf5b27ec9ddf2ae Changes include: - Flushing out the put and delete http methods - Don't store session file in /tmp, it's now up to the invoker to manage that file - Provide parameter to always delete cookies after request - Replace login / logout / if / unless parameters, though they do assist w/ a few things, as you mentioned they are not strictly needed as the builtin puppet resource dependencies can be used for that (additional parameters to test response codes and content have been added as well). At some point it might be desirable to readd these. - lots of additional testing - removed trailing whitespace, squashed commits, fixed commit message I tried removing the stubbing in the provider specs, but whenever I tried setting the parameters directly, I got an error from puppet. Looking at some of the other provider specs, there is a bit of stubbing done, so I'm not sure which way is the correct way to proceed with this. Thanks alot. ---------------------------------------- Feature #7474: puppet web resource type / provider https://projects.puppetlabs.com/issues/7474 Author: Mo Morsi Status: Accepted Priority: Normal Assignee: Category: Target version: Affected Puppet version: Keywords: puppet, web, resource, provider, type Branch: As part of the aeolus project (http://aeolusproject.org) we provide a puppet recipe to install and configure the different components that make up the framework. Several of our components primarily provide web interfaces which to configure the components and to create data, and previously we were manually 'exec'ing the curl command to invoke those requests. The problem with this is that its tricky to both verify the HTTP response code as well as parse / verify the contents of the response, so to assist with this we wrote a generic web resource type and provider (based on libcurl) to perform the actual operation. There is nothing aeolus-specific as part of this resource / provider, so I'm sharing it here with the hopes of getting some feedback and possibly for inclusion in the upstream project. The patch itself can be found here: http://git.fedorahosted.org/git/?p=aeolus/configure.git;a=commitdiff;h=24765b6d92496f6127a903c1c59e3eaf578f30a9 http://mo.morsi.org/blog/node/336 Thoughts? Any way this could be included as part of puppet? -- 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.
