Mein HTTP-Server in c ++ sendet nicht alle Dateien korrekt wieder

Question

ich auf einem HTTP-Server in c ++ bin zu arbeiten, und jetzt funktioniert es für Anfragen von Textdateien, aber wenn sie versuche, ein JPEG oder etwas zu bekommen, nur einen Teil der Datei wird gesendet. Das Problem scheint zu sein, dass, wenn ich fgets verwenden (Puffer, 2000, returned_file) die Datei Positionsanzeige zu erhöhen, scheint viel mehr, als es tatsächlich in den Puffer setzen landet. Warum sollte dies geschehen? Ich habe alle meine Code unten. Das Problem tritt in while (true) Schleife, die auftritt, wenn der Antwortcode ist 200. Vielen Dank an alle, die Antworten.

// Interpret the command line arguments
unsigned short port = 8080;

if ( (argc != 1) && (argc != 3) && (argc != 5) ) {
  cerr << "Usage: " << argv[0];
  cerr << " -p <port number> -d <base directory>" << endl;
  return 1;
}
else {
  for (int i = 1; i < argc; ++i) {
    if (strcmp(argv[i], "-p") == 0)
      port = (unsigned short) atoi(argv[++i]);
    else if (strcmp(argv[i], "-d") == 0)
      base_directory = argv[++i];
  }
}
// if base_directory was not given, set it to current working directory
if ( !base_directory ) {
  base_directory = getcwd(base_directory, 100);
}

// Create TCP socket
int tcp_sock = socket(AF_INET, SOCK_STREAM, 0);
if (tcp_sock < 0) {
  cerr << "Unable to create TCP socket." << endl;
  return 2;
}

// Create server socket
sockaddr_in server;
server.sin_family = AF_INET;
server.sin_port = htons( port );
server.sin_addr.s_addr = INADDR_ANY;

// Bind the socket
if (bind(tcp_sock, (sockaddr*)&server, sizeof(server)) < 0) {
  cerr << "Unable to bind TCP socket." << endl;
  return 3;
}

// Listen for a connection request on TCP port
listen(tcp_sock, 5);

// Create HTTP_Request object and start a while loop of accepting connections
char buffer[2000];
int bytes_recv = 0;
int recv_len = 0;
string error_reply;

HTTP_Response* response;

while (true) {
  int acc_tcp_sock = accept(tcp_sock, NULL, NULL);
  if (acc_tcp_sock == -1) {
    cerr << "Unable to open TCP connection with client." << endl;
  }
  do {
    // may want to do just one recv
    recv_len = recv( acc_tcp_sock, buffer + bytes_recv,
      2000 - bytes_recv, 0 );
    bytes_recv += recv_len;
  } while (false);
  bytes_recv = 0;
  // may want to see if this causes a memory leak
  HTTP_Request* request = HTTP_Request::Parse(buffer, 2000);

  response = handle_request(request); // function to handle the request

  // Put response header into buffer
  response->Print( buffer, 2000 );

  // if 200 and GET then send header with file
  if ( response->Get_code() == 200 ) {
    // send response header
    if ( send( acc_tcp_sock, buffer, strlen(buffer), 0 ) < 0 ) {
      cerr << "Unable to send response header to client." << endl;
    }
    if ( method == "GET" ) {
      // send file
      while ( true ) {
        fgets( buffer, 2000, returned_file );
        if ( feof( returned_file ) ) break;
        if ( send( acc_tcp_sock, buffer, strlen(buffer), 0 ) < 0 ) {
          cerr << "Unable to send file in response to client." << endl;
        }
      }
    }
    fclose( returned_file ); // close file
  }
  else {
    if ( method == "GET" ) {
      error_reply = buffer + error_page;
      if ( send( acc_tcp_sock, error_reply.c_str(), error_reply.length(), 0 ) < 0 ) {
        cerr << "Unable to send response to client." << endl;
      }
    }
    else {
      if ( send( acc_tcp_sock, buffer, strlen(buffer), 0 ) < 0 ) {
        cerr << "Unable to send respone header to client." << endl;
      }
    }
  }

  close( acc_tcp_sock ); // close the connection
}

return 0;

Answer 1

Verwenden Sie fgets() nicht binäre Daten zu lesen, die Bit-für-Bit zum Überleben braucht. Sie haben keine Aufzeichnung Scheider Übersetzung wollen, und einige Systeme kann es den Text übernehmen, wenn Sie es auf diese Weise zu lesen. Was das betrifft, sind Zeilenumbrüche und Plattenscheider völlig bedeutungslos, so dass die fgets () `Funktion des Scannens für sie ist bestenfalls eine verwirrende Ineffizienz und im schlimmsten Fall einfach nicht binär-fähig überhaupt.

Mit fread(3), oder besser noch, verwenden Sie den rohen Systemaufruf (Posix API wie auch immer, auf Nicht-Unix) lesen (2). Dies wird eine bestimmte Menge an Bit-für-Bit-Daten lesen und Ihnen sagen, wie viel es zu lesen. (In Bezug auf die binary-fähig API zu verwenden: Normalerweise werden wir beraten Daten puffern, weil wir es in der Regel in kleinen Einheiten wie Linien verarbeiten jedoch beim Verschieben eine ganze Datei von einem Ort zu einem anderen Pufferung nur. Sie verlangsamt. In diesem Fall ist es einfacher und schneller nur read() zu verwenden.)

Sie können auch nicht binäre Daten strlen(). Sie müssen nur die Byte-Zählung von dem API-Aufruf verwenden.

Answer 2

Würde mit Strlen nicht auf Binär-Dateien brechen?

send( acc_tcp_sock, buffer, strlen(buffer), 0 )

Answer 3

Es ist sehr wahrscheinlich, dass Ihre Binärdateien NULL-Bytes ('\0') enthalten. Wenn Sie Daten mit fgets lesen, kann es in den Puffer platziert werden, aber wenn man es alles übertragen, nachdem \0 (Ihr strlen Anruf sichergestellt) verloren geht.

Sie müssen also, fread verwenden, um Daten zu lesen. Es gibt eine Anzahl von Bytes, die tatsächlich gelesen wurden, so dass Sie brauchen nicht zu strlen verwenden. Und vergessen Sie nicht Datei in Binär-Modus zu öffnen!

Answer 4

Besser, um Mapping-Dateien in den Speicher lesen, auf diese Weise Sie müssen keine Puffer verwalten für Dateiinhalt zu lesen, und Sie können diesen Puffer übergeben send und Sie die Dateigröße in einer Art und Weise zu bekommen.

Wenn Sie wirklich lesen Bytes aus der Datei wollen, dann müssen Sie binäre Dateien unterscheiden zu lesen (mit dem richtigen MIME-Typ oder application/octet-stream, Bytes Speicherung mit Puffern zählen lesen) und Öffnen von Dateien als Text (text/* Mime-Typ, und Sie können Verwendung strlen).