Discussion :

[ben83510]

Bonjour,

Je essaye actuellement de coder un petit client http.J arrive a me connecter au site, j envoie ensuite une requete "GET" pour recuperer le contenue de la page mais je ne recois rien !
voici mon code:

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
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
 
#include <stdio.h>
#include <stdlib.h>
#include <winsock2.h>
#include <windows.h>
 
int main()
{
    char requete[100] = {"GET index.html HTTP/1.0"};
    char *buff;
 
    buff = calloc(512,sizeof(char));
 
    WSADATA wsa;
    WSAStartup(MAKEWORD(2, 2), &wsa);
 
    SOCKET sock;
    SOCKADDR_IN sin;
 
    sock = socket(AF_INET,SOCK_STREAM,IPPROTO_TCP);
 
    sin.sin_addr.s_addr = inet_addr("209.85.147.103"); // google.fr
    sin.sin_family = AF_INET;
    sin.sin_port = htons(80);
 
    if(connect(sock,(SOCKADDR*)&sin,sizeof(sin)) != SOCKET_ERROR)
    {
        printf("connexion ok\n");
 
        if(send(sock,requete,sizeof(requete),0) != -1)
        {
            printf("Requete envoye!\n");
        }
 
        while(recv(sock,buff,sizeof(buff),0) != SOCKET_ERROR)
        {
            printf("%s",buff);
        }
    }
 
    closesocket(sock);
 
    WSACleanup();
 
    return 0;
}

Mon programme reste bloqué sur la fonction recv.
Merci d'avance.

[ram-0000]

Code :

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

if(send(sock,requete,sizeof(requete),0) != -1)

sizeof(requete) vaut 4, il faut mettre strlen(requete)

pareil pour le recv

Code :

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

while(recv(sock,buff,sizeof(buff),0

[Obsidian]

Il faut en outre compléter ta requête avec deux retours à la ligne. Le premier pour valider la ligne que tu as écrite, et le second pour terminer ta requête par une ligne vide, nécessaire pour indiquer au serveur que tu as fini.

[ben83510]

Merci pour vos reponses.
J ai modifier en fonction de vos infos mais ca ne fonctionne toujours pas.

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
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
 
#include <stdio.h>
#include <stdlib.h>
#include <winsock2.h>
#include <windows.h>
 
int main()
{
    char requete[100] = {"GET  HTTP/1.0  \n\n"};
    char *buff;
 
    buff = calloc(512,sizeof(char));
 
    WSADATA wsa;
    WSAStartup(MAKEWORD(2, 2), &wsa);
 
    SOCKET sock;
    SOCKADDR_IN sin;
 
    sock = socket(AF_INET,SOCK_STREAM,IPPROTO_TCP);
 
    sin.sin_addr.s_addr = inet_addr("209.85.229.147"); //google.fr
    sin.sin_family = AF_INET;
    sin.sin_port = htons(80);
 
    if(connect(sock,(SOCKADDR*)&sin,sizeof(sin)) != SOCKET_ERROR)
    {
        printf("connexion ok\n");
 
        if(send(sock,requete,strlen(requete),0) != -1)
        {
            printf("Requete envoye!\n");
        }
 
        while(recv(sock,buff,strlen(buff),0) != SOCKET_ERROR)
        {
            printf("%s",buff);
        }
    }
 
    closesocket(sock);
 
    WSACleanup();
 
    return 0;
}

[ram-0000]

C'est le recv() qui va pas (faut dire que je t'ai un peu induit en erreur avec ma réponse pas tout à fait correcte).

Tu alloues un buffer de réception de 512 octets avec calloc(). Il faut dans le recv() lui dire qu'il y a 512 octets disponibles. strlen(), c'est pour le send().

Code :

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

1
2
3
4
int size_buffer_recep = 512;
char *buff = calloc(size_buffer_recep, sizeof(char));
...
while(recv(sock, buff, size_buffer_recep, 0) != SOCKET_ERROR)

Au fait, le code de retour de calloc(), cela se teste (même s'il y a peu de chance qu'il soit mauvais) autant prendre les bonnes habitudes.

[Obsidian]

Merci pour vos reponses.
J ai modifier en fonction de vos infos mais ca ne fonctionne toujours pas.

Code :

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

1
2
3
4
int main()
{
    char requete[100] = {"GET  HTTP/1.0  \n\n"};
}

Cette fois-ci, c'est la requête elle-même qui n'est plus correcte. Tu as fait sauter le / entre GET et HTTP alors qu'il indique précisément ce que tu demandes à récupérer, et les espaces surnuméraires entre 1.0 et les \n\n ne doivent pas plaire au serveur non plus.