Bug 31208 - [Qt] Pass credentials provided by XMLHTTPRequest to the network request.
Summary: [Qt] Pass credentials provided by XMLHTTPRequest to the network request.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: XML (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Linux
: P2 Normal
Assignee: Nobody
URL:
Keywords: Qt
Depends on:
Blocks: Qt46
  Show dependency treegraph
 
Reported: 2009-11-06 10:12 PST by Yael
Modified: 2009-11-16 06:53 PST (History)
5 users (show)

See Also:


Attachments
Patch (1.77 KB, patch)
2009-11-06 10:48 PST, Yael
zecke: review-
Details | Formatted Diff | Diff
Patch (2.56 KB, patch)
2009-11-09 08:01 PST, Yael
kenneth: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Yael 2009-11-06 10:12:02 PST
When XMLHTTPRequest gets credentials in the open() method, they should be passed to the network request.
Comment 1 Yael 2009-11-06 10:48:39 PST
Created attachment 42659 [details]
Patch

Follow other ports and re-attach the credentials to the request URL.
Comment 2 Holger Freyther 2009-11-07 00:40:09 PST
Comment on attachment 42659 [details]
Patch

Sorry specially as this is a regression we will need a test case for this... I don't want to see it failing again.
Comment 3 Yael 2009-11-09 06:20:10 PST
(In reply to comment #2)
> (From update of attachment 42659 [details])
> Sorry specially as this is a regression we will need a test case for this... I
> don't want to see it failing again.

Test cases are already in place for this. Please take a look at LayOutTests/http/xmlhttprequest/basic-auth.html.

Running this test, I found out that my patch fixes only async XHR, so I will submit a new patch soon, that will take care of sync XHR as well.
Comment 4 Kenneth Rohde Christiansen 2009-11-09 06:29:02 PST
Patch looks good, but should this:

// so that they can be passed to QNetworkRequest.

be

// so that they will be passed to QNetworkRequest.

?
Comment 5 Yael 2009-11-09 08:01:45 PST
Created attachment 42752 [details]
Patch

Fix authentication for sync XHR too. Also changed the comment as requested in comment #4, and added comment that LayOutTests/http/xmlhttprequest/basic-auth.html now passes.
Comment 6 WebKit Commit Bot 2009-11-09 08:08:29 PST
Comment on attachment 42752 [details]
Patch

Rejecting patch 42752 from review queue.

kenneth@webkit.org does not have reviewer permissions according to http://trac.webkit.org/browser/trunk/WebKitTools/Scripts/modules/committers.py.
Comment 7 Kenneth Rohde Christiansen 2009-11-09 08:15:41 PST
Trying again after:

Changeset [50660]  by hausmann@webkit.org

    Adding Kenneth to the reviewers list.

    * Scripts/modules/committers.py:
Comment 8 WebKit Commit Bot 2009-11-09 08:18:43 PST
Comment on attachment 42752 [details]
Patch

Rejecting patch 42752 from review queue.

kenneth@webkit.org does not have reviewer permissions according to http://trac.webkit.org/browser/trunk/WebKitTools/Scripts/modules/committers.py.
Comment 9 Yael 2009-11-09 08:34:43 PST
(In reply to comment #8)
> (From update of attachment 42752 [details])
> Rejecting patch 42752 from review queue.
> 
> kenneth@webkit.org does not have reviewer permissions according to
> http://trac.webkit.org/browser/trunk/WebKitTools/Scripts/modules/committers.py.

Congratulations Kenneth!
I think it takes a few days before a new reviewer can approve patches.
Comment 10 Kenneth Rohde Christiansen 2009-11-09 08:38:01 PST
(In reply to comment #9)
> (In reply to comment #8)
> > (From update of attachment 42752 [details] [details])
> > Rejecting patch 42752 from review queue.
> > 
> > kenneth@webkit.org does not have reviewer permissions according to
> > http://trac.webkit.org/browser/trunk/WebKitTools/Scripts/modules/committers.py.
> 
> Congratulations Kenneth!
> I think it takes a few days before a new reviewer can approve patches.

Thanks. But it is the script that checks against that file, so maybe the commit bot doesn't check again the newest version or something else is wrong. I'm trying to talk to Eric on irc.
Comment 11 Eric Seidel (no email) 2009-11-09 09:24:24 PST
Yes, sorry.  Due to bug 30084 the commit-queue requires a restart to pick up committers.py modifications.  I've restarted the queue now.
Comment 12 Kenneth Rohde Christiansen 2009-11-09 09:28:37 PST
Landed in 50662.
Comment 13 Kenneth Rohde Christiansen 2009-11-09 09:30:38 PST
OK, thanks Eric! :-) I committed one minute, before you restarted it. So I hope I was fast enough to clear the cq field.
Comment 14 Eric Seidel (no email) 2009-11-09 09:42:11 PST
You can always see what the commit-queue is currently working on via:
http://webkit-commit-queue.appspot.com/

Basically you can always outrun the commit-queue.  Its commit will fail if you manage to commit before it.