how to make QHttp work with this URL

[wysota]

Why are you complicating things so much? This should work:

void SomeClass::on_http_responseHeaderReceived( const QHttpResponseHeader &h){
  int code = h.statusCode();
  if(code>=300 && code<400 && h.hasKey("location")){
    http->get(h.value("location"));
    return;
  }
}

void SomeClass::on_http_responseHeaderReceived( const QHttpResponseHeader &h){
  int code = h.statusCode();
  if(code>=300 && code<400 && h.hasKey("location")){
    http->get(h.value("location"));
    return;
  }
}

To copy to clipboard, switch view to plain text mode 

[patrik08]

Why are you complicating things so much? This should work:

void SomeClass::on_http_responseHeaderReceived( const QHttpResponseHeader &h){
  int code = h.statusCode();
  if(code>=300 && code<400 && h.hasKey("location")){
    http->get(h.value("location"));
    return;
  }
}

void SomeClass::on_http_responseHeaderReceived( const QHttpResponseHeader &h){
  int code = h.statusCode();
  if(code>=300 && code<400 && h.hasKey("location")){
    http->get(h.value("location"));
    return;
  }
}

To copy to clipboard, switch view to plain text mode 

is ok ... if Location lowercase or uppercase ?? Location say liveheader... and if server not having session cookie....

[wysota]

It should be case insensitive. If you're really interested, take a look at QHttpHeader sources and see if value() is case sensitive.

[ber_44]

wysota, thank you, it makes the HTTP example run without error message, but the file will still not be downloaded. What else should I add to it for a correct download?

[wysota]

I can't say without knowing what's wrong. How do you handle the actual download?

[ber_44]

Well, I just took the official HTTP example by Qt, and added your code to it:

void HttpWindow::readResponseHeader(const QHttpResponseHeader &responseHeader)
{
    // handle redirections
    if (responseHeader.statusCode() >= 300 &&
        responseHeader.statusCode() < 400 &&
        responseHeader.hasKey("location")) {
        http->get(responseHeader.value("location"));
        return;
    }
 
    if (responseHeader.statusCode() != 200) 
        //...
}

void HttpWindow::readResponseHeader(const QHttpResponseHeader &responseHeader)
{
    // handle redirections
    if (responseHeader.statusCode() >= 300 &&
        responseHeader.statusCode() < 400 &&
        responseHeader.hasKey("location")) {
        http->get(responseHeader.value("location"));
        return;
    }
 
    if (responseHeader.statusCode() != 200) 
        //...
}

To copy to clipboard, switch view to plain text mode 

[wysota]

But is that branch triggered for the redirected url or not?

[ber_44]

Looks like that it gets to a redirection circle, i.e. site1 redirects to site2 which redirects to site1... I'm not sure how to work around this. Somehow it works only from browsers.

[ber_44]

Sorry, I realized that there is a direct link:
http://easynews.dl.sourceforge.net/s...ges/source.zip
It works well with this link.