Re: Make p:http-request required?

/ Jeni Tennison <jeni@jenitennison.com> was heard to say:
| Alex Milowski wrote:
|> On 4/30/07, *Norman Walsh* <ndw@nwalsh.com <mailto:ndw@nwalsh.com>> wrote:
|>     I propose that we move [p:http-request] to the "required" pile.
|>
|> + 1 to that!!!
|
| Are there any security considerations that we need to worry about?
|
| I don't really understand how p:http-request works (as in the 30th April
| draft). It doesn't seem to be a standard atomic step, since it has
| attributes that correspond to options.

Ah. True. Those things probably ought to be options.

| Shouldn't its signature be more like:
|
| <p:declare-step type="p:http-request">
|   <p:input port="request-entity" sequence="no"/>
|   <p:output port="response-status" sequence="no" />
|   <p:output port="response-headers" sequence="no"/>
|   <p:output port="response-entity" sequence="no" />
|   <p:option name="href" required="yes" />
|   <p:option name="method" required="no" value="GET" />
|   <p:option name="http-version" required="no" value="1.1" />
|   <p:param name="*" />
| </p:declare-step>
|
| possibly without the http-version option, and with other options for
| important headers such as Content-Type.

I don't think we need separate output ports for the status, headers,
and entity because the step is required to build a nice package
containing them all in one output:

<c:http-response
  status? = integer>
   (c:header*,
    c:entity?)
</c:http-response>

Similarly, the input document has to be an c:http-request, so things
like content-type can be set as headers directly in that request.

That said, I'm not wedded to it's current definition. If having
multiple inputs or outputs would be better, I could live with that.

                                        Be seeing you,
                                          norm

-- 
Norman Walsh <ndw@nwalsh.com> | As long as a word remains unspoken, you
http://nwalsh.com/            | are its master; once you utter it, you
                              | are its slave.--Solomon Ibn Gabirol