Andre Myrick - C Web Server#305
Andre Myrick - C Web Server#305AM-Myrick wants to merge 10 commits intobloominstituteoftechnology:masterfrom
Conversation
codejoncode
left a comment
There was a problem hiding this comment.
int response_length = sprintf(response, "%s\n"
"Date: %s\n"
"Connection: close\n"
"Content-Length: %d\n"
"Content-Type: %s\n"
"\n"
"%s\n",
header, buffer, content_length, content_type, new_body);
Nice work Andre. Sprintf provides a
Return Value
If successful, the total number of characters written is returned excluding the null-character appended at the end of the string, otherwise a negative number is returned in case of failure.
https://www.tutorialspoint.com/c_standard_library/c_function_sprintf.htm
codejoncode
left a comment
There was a problem hiding this comment.
sprintf(response, "%s\n"
"Date: %s\n"
"Connection: close\n"
"Content-Length: %d\n"
"Content-Type: %s\n"
"\n"
"%s\n",
header, buffer, content_length, content_type, new_body);
// Send it all!
int response_length = strlen(response);
This part of your code is only going to work for the index.html. the cat.png might have an issue with the format specifier %s.
Two ways to approach this problem memcpy or make two send 's after removing the part of your header that includes the body.
codejoncode
left a comment
There was a problem hiding this comment.
cache functions look pretty good.
codejoncode
left a comment
There was a problem hiding this comment.
void free_entry(struct cache_entry *entry)
{
///////////////////
// IMPLEMENT ME! //
///////////////////
free(entry);
}
Before your use free(entry) you will have to free every other thing you allocated space for
entry->content
entry->path
entry->content_type
No description provided.