> The canonical representation requires that we copy over the header in its
lowercase form. So, *h->key* should be populated with a lowercase version
of *header[i].key*
You're right.
I will fix this.


2013/11/30 Arvind Jayaprakash <notifications@github.com>

> In ngx_http_aws_auth.c:
>
> > +            i = 0;
> > +        }
> > +
> > +        if (header[i].hash == 0) {
> > +            continue;
> > +        }
> > +
> > +        if (ngx_strncasecmp(header[i].
> key.data, (u_char *) "x-amz-",  sizeof("x-amz-") - 1) == 0) {
> > +            h = ngx_array_push(v);
> > +            if (h == NULL) {
> > +                return NGX_ERROR;
> > +            }
> > +            h->key.data = header[i].key.data;
> > +            h->key.len  = header[i].key.len;
> > +            h->value.data  = header[i].value.data;
> > +            h->value.len  = header[i].value.len;
>
> (Answering my own question). No we cannot do it for *h->key*. The
> canonical representation requires that we copy over the header in its
> lowercase form. So, *h->key* should be populated with a lowercase version
> of *header[i].key*
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/anomalizer/ngx_aws_auth/pull/5/files#r8010322>
> .
>



-- 
Pozdrawiam
Dominik