HTTPS request returns damaged HTML body

[anda_skoa]

Also might make sense to check out QNetworkAccessManager to handle all the HTTP level (and below) protocol for you.

Cheers,
_

[ljuhrich]

NICE! It works!

Thank you very much!

both tips were very useful:

1. After I changed Accpt–encoding to none, I got some readable text, but not everything.
2. But when I changed the code using a QNetworkAcessmanager, it returned everything, and the code is a lot simpler.

void MainWindow::doRequest()
{
    // INITIALIZATION
 
    // Read username, access token
    QString id = QString(ui->edUserID->text());
    QString access_token = QString(ui->edAcessToken->text());
 
 
    manager = new QNetworkAccessManager(this);
    connect(manager, SIGNAL(finished(QNetworkReply*)), this, SLOT(resp(QNetworkReply*)));
 
    QNetworkRequest request(
                "https://graph.facebook.com/"
                + id
                + "?fields=id,name,about,sports,likes,interests&method=GET&format=json&access_token="
                + QUrl::toPercentEncoding( access_token )
                );
    request.setRawHeader("Accept-Encoding", "none");
    request.setRawHeader("User-Agent", "Mozilla/5.0 (Windows NT 5.1; rv:21.0) Gecko/20100101 Firefox/21.0");
 
    manager->get(request);
}
 
void MainWindow::resp(QNetworkReply *reply)
{
    QString responseData = QString::fromAscii(reply->readAll());
    ui->edResponse->setText( ui->edResponse->toPlainText() + "\r\n" +  responseData);
}

void MainWindow::doRequest()
{
    // INITIALIZATION

    // Read username, access token
    QString id = QString(ui->edUserID->text());
    QString access_token = QString(ui->edAcessToken->text());


    manager = new QNetworkAccessManager(this);
    connect(manager, SIGNAL(finished(QNetworkReply*)), this, SLOT(resp(QNetworkReply*)));

    QNetworkRequest request(
                "https://graph.facebook.com/"
                + id
                + "?fields=id,name,about,sports,likes,interests&method=GET&format=json&access_token="
                + QUrl::toPercentEncoding( access_token )
                );
    request.setRawHeader("Accept-Encoding", "none");
    request.setRawHeader("User-Agent", "Mozilla/5.0 (Windows NT 5.1; rv:21.0) Gecko/20100101 Firefox/21.0");

    manager->get(request);
}

void MainWindow::resp(QNetworkReply *reply)
{
    QString responseData = QString::fromAscii(reply->readAll());
    ui->edResponse->setText( ui->edResponse->toPlainText() + "\r\n" +  responseData);
}

To copy to clipboard, switch view to plain text mode 

I wish you a nice day

[anda_skoa]

Just in case: one thing people who are new to using QNetworkAccessManager can miss is that it can be used for as many requests in parallel as one would want.
I.e. in your code you create the object every time doRequest() is called, but you would only have to do it once.

Each request is paired with a reply object and QNAM is able to process them individually, even multiple requests at the same time.

Cheers,
_

[ljuhrich]

Thnk you for these nice tips, I always try to improve my skills — so every little tip helps me a lot and saves me from hours of debugging

[anda_skoa]

Well, then I have another one

You will need to delete the reply object, otherwise you are leaking it.
One way to do that is to schedule its deletion in the slot connected to the finished signals

void MainWindow::resp(QNetworkReply *reply)
{
    QString responseData = QString::fromAscii(reply->readAll());
    ui->edResponse->setText( ui->edResponse->toPlainText() + "\r\n" +  responseData);
 
    reply->deleteLater(); // schedule deletion when event loop continues event processing
}

void MainWindow::resp(QNetworkReply *reply)
{
    QString responseData = QString::fromAscii(reply->readAll());
    ui->edResponse->setText( ui->edResponse->toPlainText() + "\r\n" +  responseData);

    reply->deleteLater(); // schedule deletion when event loop continues event processing
}

To copy to clipboard, switch view to plain text mode 

Cheers,
_