Forum Discussion

BenoitB's avatar
BenoitB
Community Hero
5 years ago
Solved

GetHeader and multiple same key

I need to retrieve a cookie after a call to a Rest api. Not a problem i thought, i'll use aqHttpResponse.GetHeader("Set-Cookie"); But how to retrieve the wanted one if multple Set-Cookie are in the...
  • BenoitB's avatar
    BenoitB
    5 years ago

    So it's bullet-proof working now ...

     

    On same headers, call of extractKeyValueFromHeaders(aqHttpResponse.AllHeaders, "JSESSIONID")

    returns

    "5119F6E864B904735B8870DA08215BCD"


    Call of extractKeyValueFromHeaders(aqHttpResponse.AllHeaders, "JSESSIONID", true)

    returns

    "JSESSIONID=5119F6E864B904735B8870DA08215BCD; Path=/; Secure; HttpOnly"

     

    The revised code is (algorithm working, optimization possible) :

    function extractKeyValueFromHeaders(Headers = "", Key = "", AllValues = false) {
  if (Headers == "")
    throw Error("extractKeyValueFromHeaders() - Header is empty");
  if (Key == "")
    throw Error("extractKeyValueFromHeaders() - Key is missing");
  let headerValue = "";
  Headers = Headers.replace(/\r/g, '');
  let data = Headers.split("\n");
  let pos1, pos2, len;
  for (let i = 0; i < data.length; i++) {
    if (data[i] != "") {
      pos1 = aqString.Find(data[i], ":");
      if (pos1 != -1) {
        pos1 = aqString.Find(data[i], Key, pos1);
        if (pos1 != -1) {
          if (AllValues) {
            pos2 = data[i].length;
            len = pos1;  
          }
          else {
            pos2 = aqString.Find(data[i], ";", pos1);
            if (pos2 == -1)
              pos2 = data[i].length;
            len = pos1 + Key.length + 1;  
          }
          headerValue = data[i].substr(len, pos2 - len);
          break;
        }  
      }
    }
  }
  return headerValue;
}
AlexKaras's avatar
AlexKaras
Champion Level 3

Hi,

 

> about cleansing and tailoring of returned data

If I am correct with the expectation that

' noindex, nofollow\n\rStrict-Transport-Security: max-age=31536000'

will be returned for the X-Robots-Tag header, than this is not a cleansing case, but the case of incorrect return (more than one header is returned).

 

I would suggest to consider that by definition:

-- headers section of response is delimited by two CRLFs from the response body (if any); and

-- every header ends with one CRLF and consists of a header name, colon, space and header data (up to CRLF).

 

You may consider aqString.GetListItem() and related methods to iterate through the list of headers and then use .split(': ') to get the value.

(Or consider regular expressions... Though this may be an overkill :) )

Just my suggestion...

 

BenoitB's avatar
BenoitB
Community Hero
5 years ago

So it's bullet-proof working now ...

 

On same headers, call of extractKeyValueFromHeaders(aqHttpResponse.AllHeaders, "JSESSIONID")

returns

"5119F6E864B904735B8870DA08215BCD"


Call of extractKeyValueFromHeaders(aqHttpResponse.AllHeaders, "JSESSIONID", true)

returns

"JSESSIONID=5119F6E864B904735B8870DA08215BCD; Path=/; Secure; HttpOnly"

 

The revised code is (algorithm working, optimization possible) :

function extractKeyValueFromHeaders(Headers = "", Key = "", AllValues = false) {
  if (Headers == "")
    throw Error("extractKeyValueFromHeaders() - Header is empty");
  if (Key == "")
    throw Error("extractKeyValueFromHeaders() - Key is missing");
  let headerValue = "";
  Headers = Headers.replace(/\r/g, '');
  let data = Headers.split("\n");
  let pos1, pos2, len;
  for (let i = 0; i < data.length; i++) {
    if (data[i] != "") {
      pos1 = aqString.Find(data[i], ":");
      if (pos1 != -1) {
        pos1 = aqString.Find(data[i], Key, pos1);
        if (pos1 != -1) {
          if (AllValues) {
            pos2 = data[i].length;
            len = pos1;  
          }
          else {
            pos2 = aqString.Find(data[i], ";", pos1);
            if (pos2 == -1)
              pos2 = data[i].length;
            len = pos1 + Key.length + 1;  
          }
          headerValue = data[i].substr(len, pos2 - len);
          break;
        }  
      }
    }
  }
  return headerValue;
}
  • AlexKaras's avatar
    AlexKaras
    Champion Level 3
    5 years ago

    > extractKeyValueFromHeaders(aqHttpResponse.AllHeaders, "JSESSIONID")

    Ah, now I got your logic... OK then :)

     

    • BenoitB's avatar
      BenoitB
      Community Hero
      5 years ago

      I noticed that u didnt see my need (only value of the key itself) but thanks to u i improved the code to be more generic..

      This is good team work ;)

       

      So do u agree on my last proposal ?

      • AlexKaras's avatar
        AlexKaras
        Champion Level 3
        5 years ago

        > do u agree on my last proposal ?

        Yep :)