Discussion :

[gregb34]

Bonjour,

Je suis en train de créer un petit serveur HTTP et j'ai un probleme lors de la lecture des requetes des clients...

J'ouvre bien un thread a la connexion d'un client, dans ce thread, je dois lire la requete effectué par le navigateur.

Je fais donc un :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
    char buffer[100];
    int lecture;
 
    do{
        lecture = read( _fdClient, buffer, sizeof(buffer));
        if ((int)sizeof(buffer)>lecture){
            buffer[lecture] = 0;
        }
        cout << buffer << endl;
    }
    while(lecture>0);
 
    cout <<"Fin de la requete"<<endl;

"Le fin de requete" ne s'affiche jamais !!! Pourtant quand le serveur a tout lu, le read devrait me renvoyer 0 et ce n'est pas le cas. Une idée ?

Merci de votre aide.

PS : Apres un peu plus de recherche, il semblerait que le read soit bloquant, ne souhaitant pas rendre mon socket non bloquant que faire pour contourner ce soucis ?

Je pourrais tres bien mettre un char buffer[4096] pour tout lire d'un coup et encore, si la requete est plus grande, je peux pas le gérer....

Une solution pour gérer tous les cas ? MERCI à vous.

[ram-0000]

Je pense qu'il te manque le traitement de fermeture du socket

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
for(;;)
{
char buffer[100];
   int lecture = read( _fdClient, buffer, sizeof(buffer));
   if(lecture == 0)
   {
      // erreur de lecture
      break;
   }
   if(lecture == -1)
   {
      // autre erreur de lecture
      break;
   }
 
   if((int)sizeof(buffer)>lecture)
   {
      // pourquoi pas mais si tu lis exactement 100 octets
      // le caractère 0 final ne sera pas positionné
      // et tu risques d'afficher n'importe quoi
      buffer[lecture] = 0;
   }
 
   // traitement du buffer recu
   cout << buffer << endl;
}
 
// traitement de la fin de la requete
cout <<"Fin de la requete"<<endl;

[Médinoc]

Note: Si jamais tu veux être compatible avec Windows, utilise recv() plutôt que read() sur un socket. Si ton programme est seulement destiné aux systèmes POSIX, tu peux garder read().

[gregb34]

Merci pour vos précisions, cepdant, cela ne résout pas mon problème.

Lorsque j'ai lu toutes les données, le read n'a plus rien à lire est devient bloquant en attendant de nouvelles données à lire.

Je crois que je vais être obligé d'enlever la boucle et de tout lire d'un coup en limitant les requetes a 4096 octets.

[loufoque]

Ça te renvoit 0 quand la socket est fermée.
Sauf que bon, si tu fais un serveur HTTP, le client t'envoie les données, tu les lis, tu écris la réponse, et ensuite tu fermes la socket (ou pas, si tu fais du keep-alive).

Bref, lis la RFC avant de vouloir implémenter un protocole...

[gregb34]

Je lis les données et je lui réponds : OK

Tu as lu mon problème et mes hypotheses non ? Comment veux tu que j'emette une réponse si je reste bloqué par le read() ?

[loufoque]

Il faut que tu apprennes comment HTTP fonctionne.
Pour savoir où le contenu en entrée se termine, il faut lire l'entête "Content-Length" ou décoder le "Transfer-Encoding: chunked".
(Le client doit d'ailleurs faire la même chose en keep-alive)

Vouloir écrire un serveur HTTP alors que tu n'as aucune compréhension de comment fonctionne celui-ci, ou pire, de comment fonctionnent les sockets, est aberrant.

[gregb34]

J'ai pas la prétention de vouloir ecrire Apache, c'est juste un projet scolaire... Faut se calmer...

Quand j'affiche la requete d'un client qui se connecte, j'ai que ça, je vois pas les informations dont tu me parles.

GET / HTTP/1.1
Host: localhost:8888
User-Agent: Mozilla/5.0 (X11; U; Linux i686; fr; rv:1.8.1.13) Gecko/20080325 Ubuntu/7.10 (gutsy) Firefox/2.0.0.13
Accept: text/xml,application/xml,application/xhtml+xml,text/html;q=0.9,text/plain;q=0.8,image/png,*/*;q=0.5
Accept-Language: fr-fr,fr;q=0.5
Accept-Encoding: gzip,deflate
Accept-Charset: ISO-8859-1,utf-8;q=0.7,*;q=0.7
Keep-Alive: 300
Connection: keep-alive

A noter que je l'affiche avec un seul read et un buffer[4096] ce qui est lobjet de mon probleme pour l'instant.

[loufoque]

C'est une requête sans donnée en entrée, comme d'ailleurs toute requête GET. Les données en entrée c'est essentiellement pour POST (PUT et OPTIONS aussi, mais tu t'en fous, tu vas sûrement pas les implémenter).
Ça ne fournit ici que des entêtes, comme tu peux le constater.

Le format c'est
TypeDeRequete Request-URI HTTP/Version<CR><LF>
Entete-1: valeur1<CR><LF>
Entete-2: valeur2<CR><LF>
...
Entete-n: valeurn<CR><LF>
<CR><LF>
<données, le cas échéant>
<fin de transmission>

pour le retour, la première ligne change, elle indique un code de retour. (HTTP/Version Code Texte<CR><LF> il me semble)

Quand il y a des données, les entêtes doivent donner son type, et éventuellement la taille des données, pour savoir jusqu'où lire avant de répondre. Si la taille n'est pas fournie, le chunked doit être utilisé.
À noter qu'il n'y a pas de chunked en HTTP 1.0.

Et ce, que ce soit pour l'entrée ou la sortie.
Tout cela est clairement expliqué dans la RFC (partie 4), que tu devrais connaître presque par cœur si tu veux faire un serveur HTTP.

Je te donne le lien, au cas où t'aurais du mal :
http://www.faqs.org/rfcs/rfc2616.html