anomalizer / ngx_aws_auth

nginx module to proxy to authenticated AWS services
BSD 2-Clause "Simplified" License
470 stars 144 forks source link

add functionality for all request #5

Closed tarfik closed 10 years ago

tarfik commented 10 years ago

Hi, I added signing request if it is s3 style (with x-amz-..., content-md5, content-type and Date headers). Add to config is "proxy_set_header x-amz-date $aws_date;" enable add x-amz-date to string_to_sign.

Regards Dominik

anomalizer commented 10 years ago

Can you provide an example of how 57ce013 would be useful?

tarfik commented 10 years ago

location ~ ^/([^/])(/.)$ { set $bucket $1; set $filename $2; s3_bucket "awsbucket-${bucket}"; chop_prefix "/${bucket}";

eg. request -> /testbucket/testfile.txt -> In previous implementation inside module variable s3_bucket had string "awsbucket-${bucket}" After this change -> s3_bucket = "awsbucket-testbucket". The same for chop_prefix.

2013/11/30 Arvind Jayaprakash notifications@github.com

Can you provide an example of how 57ce013https://github.com/anomalizer/ngx_aws_auth/commit/57ce013would be useful?

— Reply to this email directly or view it on GitHubhttps://github.com/anomalizer/ngx_aws_auth/pull/5#issuecomment-29547369 .

Pozdrawiam Dominik

anomalizer commented 10 years ago

@tarfik , would you be providing the updates?

tarfik commented 10 years ago

Sorry, I have a little vacation. I'll make a commit changes in the next few days On Dec 28, 2013 5:20 AM, "Arvind Jayaprakash" notifications@github.com wrote:

@tarfik https://github.com/tarfik , would you be providing the updates?

— Reply to this email directly or view it on GitHubhttps://github.com/anomalizer/ngx_aws_auth/pull/5#issuecomment-31289999 .

tarfik commented 10 years ago

I have commited suggested changes.