[privoxy-users] [ ijbswa-Support Requests-1464853 ] Changing Referer header reports an error in logs

[SourceForge.net <no...@so...>]

Support Requests item #1464853, was opened at 04/05/06 04:54
Message generated for change (Comment added) made by sf-robot
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=211118&aid=1464853&group_id=11118

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
>Status: Closed
Priority: 5
Submitted By: Nobody/Anonymous (nobody)
Assigned to: David Schmidt (david__schmidt)
Summary: Changing Referer header reports an error in logs

Initial Comment:
OS WXP SP2
Proxy Version 3.0.3

"+hide-referer{string_string} \" report an error in logs instead of changing Referer header. Although, "+hide-referer{http://somesite.com} \" changes header properly. Is "http://" required for any string? There is no info about this in manual.

----------------------------------------------------------------------

>Comment By: SourceForge Robot (sf-robot)
Date: 04/20/06 19:20

Message:
Logged In: YES 
user_id=1312539

This Tracker item was closed automatically by the system. It was
previously set to a Pending status, and the original submitter
did not respond within 14 days (the time period specified by
the administrator of this Tracker).

----------------------------------------------------------------------

Comment By: David Schmidt (david__schmidt)
Date: 04/06/06 11:36

Message:
Logged In: YES 
user_id=249980

Sometimes the source code is the best manual. :-)  There are
some words of wisdom here:
http://www.privoxy.org/3.0.3/user-manual/actions-file.html#HIDE-REFERRER

Truncated souce from parsers.c follows - it is indeed
looking for null, "block", "forge", or the string to start
with "http://".

   newval = csp->action->string[ACTION_STRING_REFERER];

   if ((newval == NULL) || (0 == strcmpic(newval, "block")) )
   {
    ...
   }
   else if (0 == strncmpic(newval, "http://", 7))
   {
      /*
       * We have a specific (fixed) referer we want to send.
       */
      ...
   }
   else
   {
      /*
       * Forge a referer as http://[hostname:port of REQUEST]/
       * to fool stupid checks for in-site links
       */
      if (0 != strcmpic(newval, "forge"))
      {
       ...
      }

      *header = strdup("Referer: http://");
   }


----------------------------------------------------------------------

Comment By: Fabian Keil (fabiankeil)
Date: 04/05/06 07:00

Message:
Logged In: YES 
user_id=875547

At the moment the string has to begin with "http://".

If you don't like that, change line 1146
in parsers.c from

else if (0 == strncmpic(newval, "http://", 7))

to

else if (0 != strcmpic(newval, "forge"))

Fabian

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=211118&aid=1464853&group_id=11118