search

DMTF / Redfish-Protocol-Validator

The Redfish Protocol Validator tests the HTTP protocol behavior of a Redfish service to validate that it conforms to the Redfish specification.
Other
14 stars 12 forks source link

Regarding the specification of AL #54

Closed FelixD999 closed 1 year ago

FelixD999 commented 1 year ago 
          @FelixD999 I'm looking at the AL header test a bit more and I noticed that it looks like the contents of your header appear to be wrapped in double quotes; this is not expected. The test is examining the "path" portion of the URL and ignoring the rest.

Originally posted by @mraineri in https://github.com/DMTF/Redfish-Protocol-Validator/issues/51#issuecomment-1398506972

Hi, After verified, the content of the AL header indeed be wrapped in double quotes. AL_header

I want to ask if there is an opportunity to optimize the tool so that the test process can skip the quotes. Refered the UPnP Device Architecture 2.0, but there seems to be no specification about the type of AL mentioned.

Thanks, Felix

mraineri commented 1 year ago

All examples I've been finding for AL do not use double quotes. It doesn't seem to be common in other HTTP headers either to wrap string values like that in double quotes (like the Location header). It seems to be generally bad practice unless the encoding of the value requires double quotes for a particular header (like with ETags).

FelixD999 commented 1 year ago

Got it, thank you very much for your reply.

mraineri commented 1 year ago

Closing; no plans on changing the tool since we're not expecting double quotes in the header. Please re-open if more discussion about this is needed.