cos-cpp-sdk-v5 icon indicating copy to clipboard operation
cos-cpp-sdk-v5 copied to clipboard

fix bug: not support lowercase keys in response header

Open zhoumhh opened this issue 4 years ago • 0 comments

void BaseResp::ParseFromHeaders(const std::map<std::string, std::string>& headers)

In this function, parameter headers is a map, keys in it maybe full lowercase in some environment, such as:

headers = {
    { "content-length": "5" },
    { "content-type": "text/plain" }
}

But variables (kReqHeader*) is defined by initial capitalization in "cos_params.h", such as:

const std::string kReqHeaderContentLen = "Content-Length";
const std::string kReqHeaderContentType = "Content-Type";

This function couldn't parse header correctly when response header is full lowercase!

The code added will find the value in map by lowercase key again.

zhoumhh avatar Aug 02 '19 07:08 zhoumhh