Work at SourceForge, help us to make it a better place! We have an immediate need for a Support Technician in our San Francisco or Denver office.

Close

#24 CDDB HTTP request fixes

closed
None
5
2006-12-13
2006-08-01
Anonymous
No

Hi!,

I found a two small problems in
CDdb::SendRemoteMessage, for which I'm submitting this
patch.

1. The "User-Agent" header was misspelt "UserAgent"
(needs an hyphen according to RFC2616). I also changed
the RFC822 reference to the correct RFC2616.

2. There was no "Host" header being sent. This one is
required when accessing a virtual host (a domain at an
IP that handles several domains).

I'm attaching a diff of these changes.

Discussion

  • diff of suggested changes to Cddb.cpp

     
    Attachments
  • chipple
    chipple
    2006-08-01

    Logged In: YES
    user_id=1565946

    This patch was submitted by me (I forgot to login first).

     
  • Albert L Faber
    Albert L Faber
    2006-12-13

    Logged In: YES
    user_id=1069
    Originator: NO

    thanks for the patch, patch has been applied

     
  • Albert L Faber
    Albert L Faber
    2006-12-13

    • assigned_to: nobody --> afaber
    • status: open --> closed