[Groonga-commit] groonga/gcs [master] Accept multiple priviledged IP ranges correctly

Back to archive index

null+****@clear***** null+****@clear*****
2012年 8月 2日 (木) 18:39:22 JST


SHIMODA Hiroshi	2012-08-02 18:39:22 +0900 (Thu, 02 Aug 2012)

  New Revision: 171618bbbee47eb97a88d2b671a327cd9ec8cc6b
  https://github.com/groonga/gcs/commit/171618bbbee47eb97a88d2b671a327cd9ec8cc6b

  Log:
    Accept multiple priviledged IP ranges correctly

  Modified files:
    lib/api/2011-02-01/configuration.js

  Modified: lib/api/2011-02-01/configuration.js (+4 -3)
===================================================================
--- lib/api/2011-02-01/configuration.js    2012-08-02 18:37:27 +0900 (cc33477)
+++ lib/api/2011-02-01/configuration.js    2012-08-02 18:39:22 +0900 (2095f90)
@@ -330,14 +330,15 @@ function getClientIp(request) {
 
 
 exports.createHandler = function(context, config) {
-  var privilegedRanges = config && config.privilegedRanges;
-  privilegedRanges = privilegedRanges.split(/[,\| ]/);
+  var privilegedRanges = config &&
+                         config.privilegedRanges &&
+                         config.privilegedRanges.split(/[,\| ]/);
   return function(request, response, next) {
     var message, body;
 
     // GCS specific behaviour: prevent to access this API from specific IP
     // range.
-    if (privilegedRange) {
+    if (privilegedRanges && privilegedRanges.length) {
       if (!privilegedRanges.some(function(privilegedRange) {
             return ipv4.isInRange(getClientIp(request), privilegedRange);
           })) {
-------------- next part --------------
HTML$B$NE:IU%U%!%$%k$rJ]4I$7$^$7$?(B...
Download 



Groonga-commit メーリングリストの案内
Back to archive index