[6/7] qmgr: Add support for Basic authentication.

[Jacek Caban]

Hi Hans,

On 06/08/15 14:32, Hans Leidekker wrote:
>  static HRESULT WINAPI http_negotiate_BeginningTransaction(
>      IHttpNegotiate *iface, LPCWSTR url, LPCWSTR headers, DWORD reserved, LPWSTR *add_headers)
>  {
>      DLBindStatusCallback *callback = impl_from_IHttpNegotiate(iface);
> -    FIXME("(%p)->(%s %s %u %p)\n", callback, debugstr_w(url), debugstr_w(headers), reserved, add_headers);
> -    return E_NOTIMPL;
> +    BackgroundCopyJobImpl *job = callback->file->owner;
> +    WCHAR *str, *auth_header;
> +    DWORD len;
> +
> +    TRACE("(%p)->(%s %s %d %p)\n", callback, debugstr_w(url), debugstr_w(headers), reserved, add_headers);
> +
> +    if ((auth_header = build_auth_header(&job->http_options.creds)))
> +    {
> +        if (!job->http_options.headers)
> +        {
> +            *add_headers = auth_header;
> +            return S_OK;
> +        }
> +        else
> +        {
> +            len = strlenW(auth_header) + strlenW(job->http_options.headers);
> +            if (!(str = CoTaskMemAlloc((len + 1) * sizeof(WCHAR))))
> +            {
> +                CoTaskMemFree(auth_header);
> +                return E_OUTOFMEMORY;
> +            }
> +            strcpyW(str, auth_header);
> +            strcatW(str, job->http_options.headers);
> +            CoTaskMemFree(auth_header);
> +            *add_headers = str;
> +            return S_OK;
> +        }
> +    }
> +    else if (job->http_options.headers)
> +    {
> +        if (!(*add_headers = co_strdupW(job->http_options.headers))) return E_OUTOFMEMORY;
> +        return S_OK;
> +    }
> +
> +    *add_headers = NULL;
> +    return S_OK;

It doesn't sound like a good idea to manually handle that here. We
should most likely use IAuthenticate (although it's not yet supported in
urlmon).

Jacek