Opened on 08/25/2014 at 08:01:39 AM

Closed on 11/05/2014 at 04:56:10 PM

Last modified on 12/11/2014 at 02:02:11 PM

#1265 closed change (fixed)

Figure out a method to detect the type of the request correctly

Reported by: oleksandr Assignee: sergz
Priority: P2 Milestone: Adblock-Plus-for-Internet-Explorer-1.3
Module: Adblock-Plus-for-Internet-Explorer Keywords:
Cc: oleksandr, mapx, arthur Blocked By:
Blocking: #1183, #1210, #1218, #1231, #1242 Platform: Internet Explorer
Ready: no Confidential: no
Tester: Verified working: no
Review URL(s):



Currently we try to retrieve the mime type from the request and assume the type based on that. Unfortunately mime type is very often unavailable. In that case we detect the type of the request based on the extension.

What to change

Find a way to always have a mime type of a request, or try to figure out another way of detecting the type of the request.

Attachments (0)

Change History (11)

comment:1 Changed on 08/25/2014 at 08:01:57 AM by oleksandr

  • Platform changed from Unknown to Internet Explorer

comment:2 Changed on 08/25/2014 at 08:07:55 AM by oleksandr

  • Blocking 1218, 1231, 1242 added

comment:3 Changed on 08/25/2014 at 08:35:22 AM by mapx

  • Cc mapx added

comment:4 Changed on 08/25/2014 at 08:43:33 AM by oleksandr

  • Milestone Adblock-Plus-for-Internet-Explorer-next deleted

comment:5 Changed on 09/05/2014 at 04:00:34 PM by sergz

  • Owner changed from oleksandr to sergz

We can obtain Accept header in BeginningTransaction and make a decision here, if it's blocked we should return E_ABORT. It's possible via querying for IWinInetHttpInfo from the target protocol and calling QueryInfo(HTTP_QUERY_RAW_HEADERS_CRLF | HTTP_QUERY_FLAG_REQUEST_HEADERS...) . The latter call fails when it's done from our OnStart method, it seems that original target protocol::Start prepares the request and headers and then calls BeginningTransaction.

I'm going to refactor OnStart, BeginningTransaction and WBPassthruSink to deal with it.

comment:6 Changed on 09/08/2014 at 07:57:15 AM by arthur

  • Cc arthur added

comment:7 Changed on 09/11/2014 at 08:30:54 AM by sergz

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

comment:8 Changed on 10/07/2014 at 01:50:31 PM by oleksandr

  • Priority changed from P1 to P2

comment:9 Changed on 10/24/2014 at 10:18:43 AM by oleksandr

  • Review URL(s) modified (diff)

comment:10 Changed on 11/05/2014 at 04:56:10 PM by oleksandr

  • Resolution set to fixed
  • Status changed from reviewing to closed

comment:11 Changed on 12/11/2014 at 02:02:11 PM by oleksandr

  • Milestone set to Adblock-Plus-for-Internet-Explorer-next

Add Comment

Modify Ticket

Change Properties
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from sergz.
Note: See TracTickets for help on using tickets.