how to add a module to decode/modify parameter on HTTP request

2014-04-29 Thread ????
Hello,

We are new on Nginx module development. Currently we encounter an issue as 
below. Could you please help to check? Thanks in advanced.

Scenario:

add a module to decode/modify parameter on HTTP request. Then pass the updated 
HTTP request to PHP to be handled as normal. 

Question:

How to modify nginx.conf and how to decode/modify HTTP request?



Currently, all examples about nginx module I got are nginx modules handle HTTP 
request and then provide response to client directly.

nginx.conf

   ...



location ~ \.php$ {

fastcgi_split_path_info ^(.+\.php)(/.+)$;

# NOTE: You should have cgi.fix_pathinfo = 0; 
in php.ini

   

# With php5-cgi alone:

fastcgi_pass 127.0.0.1:9000;

# With php5-fpm:

fastcgi_pass unix:/var/run/php5-fpm.sock;

fastcgi_index index.php;

include fastcgi_params;

}___
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel

Re: [nginx] Adding Support for Weak ETags

2014-04-29 Thread Aaron Peschel
Hello Maxim,

If you provide a copy of your newer draft patch, I am willing to spend
time helping improve it as you see fit.

-Aaron Peschel

On Fri, Apr 25, 2014 at 4:35 AM, Maxim Dounin mdou...@mdounin.ru wrote:
 Hello!

 On Thu, Apr 24, 2014 at 06:20:24PM -0700, Aaron Peschel wrote:

 Hi Maxim,

 Is the draft patch the same as the one that your posted in the
 previous thread, or has more work been done since then?

 The one I've posted is to ignore weak etags.  The draft one is to
 downgrade strict etags to weak etags.


 -Aaron Peschel

 On Mon, Apr 21, 2014 at 5:26 AM, Maxim Dounin mdou...@mdounin.ru wrote:
  Hello!
 
  On Thu, Apr 17, 2014 at 05:39:40PM -0700, Aaron Peschel wrote:
 
  Hello,
 
  I am interested in getting support for Weak ETags into the mainline.
  There was some discussion previously in here previously that developed
  a quick patch to add support. What additional functionality would be
  required and what steps should be followed to get weak etag
  functionality added to nginx? I am willing to do the work, I just need
  some help with heading in the right direction.
 
  I had a quick draft patch sitting in my patchqueue since previous
  discussion (see [1]) to downgrade strict etags to weak ones.  It
  needs more work though, as I'm not yet happy with the code.  I
  hope I'll be able to find some time and finish it in 1.7.x.
 
  [1] 
  http://mailman.nginx.org/pipermail/nginx-devel/2013-November/004523.html
 
  --
  Maxim Dounin
  http://nginx.org/
 
  ___
  nginx-devel mailing list
  nginx-devel@nginx.org
  http://mailman.nginx.org/mailman/listinfo/nginx-devel

 ___
 nginx-devel mailing list
 nginx-devel@nginx.org
 http://mailman.nginx.org/mailman/listinfo/nginx-devel

 --
 Maxim Dounin
 http://nginx.org/

 ___
 nginx-devel mailing list
 nginx-devel@nginx.org
 http://mailman.nginx.org/mailman/listinfo/nginx-devel

___
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel


Re: [nginx] Adding Support for Weak ETags

2014-04-29 Thread Adam Arsenault
Hi Maxim/Aaron,

Would love to see support for Weak Etags in nginx. The lack of support for
gzip + etags has been a major problem for us as we use etags for caching
(and need gzip to work with it) in a bunch of different place in our
application.

Let me know if there is anything I can do to help out here as well.

Thanks,
Adam Arsenault


On Tue, Apr 29, 2014 at 11:47 AM, Aaron Peschel aaron.pesc...@gmail.comwrote:

 Hello Maxim,

 If you provide a copy of your newer draft patch, I am willing to spend
 time helping improve it as you see fit.

 -Aaron Peschel

 On Fri, Apr 25, 2014 at 4:35 AM, Maxim Dounin mdou...@mdounin.ru wrote:
  Hello!
 
  On Thu, Apr 24, 2014 at 06:20:24PM -0700, Aaron Peschel wrote:
 
  Hi Maxim,
 
  Is the draft patch the same as the one that your posted in the
  previous thread, or has more work been done since then?
 
  The one I've posted is to ignore weak etags.  The draft one is to
  downgrade strict etags to weak etags.
 
 
  -Aaron Peschel
 
  On Mon, Apr 21, 2014 at 5:26 AM, Maxim Dounin mdou...@mdounin.ru
 wrote:
   Hello!
  
   On Thu, Apr 17, 2014 at 05:39:40PM -0700, Aaron Peschel wrote:
  
   Hello,
  
   I am interested in getting support for Weak ETags into the mainline.
   There was some discussion previously in here previously that
 developed
   a quick patch to add support. What additional functionality would be
   required and what steps should be followed to get weak etag
   functionality added to nginx? I am willing to do the work, I just
 need
   some help with heading in the right direction.
  
   I had a quick draft patch sitting in my patchqueue since previous
   discussion (see [1]) to downgrade strict etags to weak ones.  It
   needs more work though, as I'm not yet happy with the code.  I
   hope I'll be able to find some time and finish it in 1.7.x.
  
   [1]
 http://mailman.nginx.org/pipermail/nginx-devel/2013-November/004523.html
  
   --
   Maxim Dounin
   http://nginx.org/
  
   ___
   nginx-devel mailing list
   nginx-devel@nginx.org
   http://mailman.nginx.org/mailman/listinfo/nginx-devel
 
  ___
  nginx-devel mailing list
  nginx-devel@nginx.org
  http://mailman.nginx.org/mailman/listinfo/nginx-devel
 
  --
  Maxim Dounin
  http://nginx.org/
 
  ___
  nginx-devel mailing list
  nginx-devel@nginx.org
  http://mailman.nginx.org/mailman/listinfo/nginx-devel

 ___
 nginx-devel mailing list
 nginx-devel@nginx.org
 http://mailman.nginx.org/mailman/listinfo/nginx-devel




-- 
[image: HootSuite]

*Adam Arsenault*
Senior Software Engineer, Mobile Web  |  HootSuite
@Adam_Arsenault http://twitter.com/Adam_Arsenault |
hootsuitehttp://www.hootsuite.com/
 | blog http://blog.hootsuite.com/ | facebookhttp://facebook.com/hootsuite
___
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel

[Patch] OCSP stapling: missing OCSP request free call

2014-04-29 Thread Filipe Da Silva
Hello,

I think there is a free call to do when the OCSP request is well
created, and encoded.
Or did I miss something ?

Regards,
Filipe


# HG changeset patch
# User Filipe da Silva fdasilv...@gmail.com
# Date 1398802958 -7200
#  Tue Apr 29 22:22:38 2014 +0200
# Node ID 8b75fcb0055f448cd5fc4f922b5b7457b38754c4
# Parent  3a48775f1535fe37cd9c034d92c5a5e9ae208c1e
OCSP stapling: missing OCSP request free call

diff -r 3a48775f1535 -r 8b75fcb0055f src/event/ngx_event_openssl_stapling.c
--- a/src/event/ngx_event_openssl_stapling.cTue Apr 29 12:28:41 2014 +0400
+++ b/src/event/ngx_event_openssl_stapling.cTue Apr 29 22:22:38 2014 +0200
@@ -1195,6 +1195,8 @@ ngx_ssl_ocsp_create_request(ngx_ssl_ocsp
 b-last = p;
 ctx-request = b;

+OCSP_REQUEST_free(ocsp);
+
 return NGX_OK;

 failed:

___
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel