BaseHttpTransaction::SetPayload(...) overloads are almost exact duplicates. This needs to be cleaned up. The code duplication comes from a lack of one or more member functions which are needed to verify the payload body contents (looking for and adding, if needed, terminating CRLFs), and adjust payload size members etc. If such function(s) were added to BaseHttpTransaction, this duplication could be entirely removed.
BaseHttpTransaction::SetPayload(...)overloads are almost exact duplicates. This needs to be cleaned up. The code duplication comes from a lack of one or more member functions which are needed to verify the payload body contents (looking for and adding, if needed, terminating CRLFs), and adjust payload size members etc. If such function(s) were added toBaseHttpTransaction, this duplication could be entirely removed.