httpd-cvs mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bnicho...@apache.org
Subject svn commit: r360207 - in /httpd/httpd/branches/authz-dev: include/http_core.h server/core.c
Date Sat, 31 Dec 2005 00:28:48 GMT
Author: bnicholes
Date: Fri Dec 30 16:28:46 2005
New Revision: 360207

URL: http://svn.apache.org/viewcvs?rev=360207&view=rev
Log:
Remove the 'satisfy' directive and the ap_satisfy() and ap_requires() APIs since they no longer
serve a purpose

Modified:
    httpd/httpd/branches/authz-dev/include/http_core.h
    httpd/httpd/branches/authz-dev/server/core.c

Modified: httpd/httpd/branches/authz-dev/include/http_core.h
URL: http://svn.apache.org/viewcvs/httpd/httpd/branches/authz-dev/include/http_core.h?rev=360207&r1=360206&r2=360207&view=diff
==============================================================================
--- httpd/httpd/branches/authz-dev/include/http_core.h (original)
+++ httpd/httpd/branches/authz-dev/include/http_core.h Fri Dec 30 16:28:46 2005
@@ -303,15 +303,17 @@
  *      SATISFY_ALL    -- all of the requirements must be met.
  *      SATISFY_NOSPEC -- There are no applicable satisfy lines
  * </pre>
- */
-AP_DECLARE(int) ap_satisfies(request_rec *r);
+ *
+AP_DE CLARE(int) ap_satisfies(request_rec *r);
+*/
 
 /**
  * Retrieve information about all of the requires directives for this request
  * @param r The current request
  * @return An array of all requires directives for this request
- */
-AP_DECLARE(const apr_array_header_t *) ap_requires(request_rec *r);    
+ *
+AP_DE CLARE(const apr_array_header_t *) ap_requires(request_rec *r);    
+*/
 
 #ifdef CORE_PRIVATE
 
@@ -451,12 +453,12 @@
     
     char *ap_default_type;
   
-    /* Authentication stuff.  Groan... */
-    
-    int *satisfy; /* for every method one */
-    char *ap_auth_type; /* Deprecated see mod_authn */
-    char *ap_auth_name; /* Deprecated see mod_authn */
-    apr_array_header_t *ap_requires; /* Deprecated see mod_authz */
+//    /* Authentication stuff.  Groan... */
+//    
+//    int *satisfy; /* for every method one */
+//    char *ap_auth_type; /* Deprecated see mod_authn */
+//    char *ap_auth_name; /* Deprecated see mod_authn */
+//    apr_array_header_t *ap_requires; /* Deprecated see mod_authz */
 
     /* Custom response config. These can contain text or a URL to redirect to.
      * if response_code_strings is NULL then there are none in the config,
@@ -685,8 +687,9 @@
  * authorization values with mod_authz_host
  */
 
-APR_DECLARE_OPTIONAL_FN(const apr_array_header_t *, authz_ap_requires,
+/*APR_DECLARE_OPTIONAL_FN(const apr_array_header_t *, authz_ap_requires,
                         (request_rec *r));
+*/                        
 APR_DECLARE_OPTIONAL_FN(int, authz_some_auth_required, (request_rec *r));
 APR_DECLARE_OPTIONAL_FN(const char *, authn_ap_auth_type, (request_rec *r));
 APR_DECLARE_OPTIONAL_FN(const char *, authn_ap_auth_name, (request_rec *r));

Modified: httpd/httpd/branches/authz-dev/server/core.c
URL: http://svn.apache.org/viewcvs/httpd/httpd/branches/authz-dev/server/core.c?rev=360207&r1=360206&r2=360207&view=diff
==============================================================================
--- httpd/httpd/branches/authz-dev/server/core.c (original)
+++ httpd/httpd/branches/authz-dev/server/core.c Fri Dec 30 16:28:46 2005
@@ -99,7 +99,7 @@
 static void *create_core_dir_config(apr_pool_t *a, char *dir)
 {
     core_dir_config *conf;
-    int i;
+/*    int i;*/
 
     conf = (core_dir_config *)apr_pcalloc(a, sizeof(core_dir_config));
 
@@ -118,10 +118,12 @@
     conf->use_canonical_phys_port = USE_CANONICAL_PHYS_PORT_UNSET;
 
     conf->hostname_lookups = HOSTNAME_LOOKUP_UNSET;
+/*
     conf->satisfy = apr_palloc(a, sizeof(*conf->satisfy) * METHODS);
     for (i = 0; i < METHODS; ++i) {
         conf->satisfy[i] = SATISFY_NOSPEC;
     }
+*/    
 
 #ifdef RLIMIT_CPU
     conf->limit_cpu = NULL;
@@ -347,7 +349,7 @@
      */
 
     /* use a separate ->satisfy[] array either way */
-    conf->satisfy = apr_palloc(a, sizeof(*conf->satisfy) * METHODS);
+/*    conf->satisfy = apr_palloc(a, sizeof(*conf->satisfy) * METHODS);
     for (i = 0; i < METHODS; ++i) {
         if (new->satisfy[i] != SATISFY_NOSPEC) {
             conf->satisfy[i] = new->satisfy[i];
@@ -355,6 +357,7 @@
             conf->satisfy[i] = base->satisfy[i];
         }
     }
+*/    
 
     if (new->server_signature != srv_sig_unset) {
         conf->server_signature = new->server_signature;
@@ -732,7 +735,7 @@
 
 /*
  * Optional function coming from mod_ident, used for looking up ident user
- */
+ *
 static APR_OPTIONAL_FN_TYPE(authz_ap_requires) *authz_ap_requires;
 
 AP_DECLARE(const apr_array_header_t *) ap_requires(request_rec *r)
@@ -742,7 +745,9 @@
     }
     return NULL;
 }
+*/
 
+/*
 AP_DECLARE(int) ap_satisfies(request_rec *r)
 {
     core_dir_config *conf;
@@ -752,6 +757,7 @@
 
     return conf->satisfy[r->method_number];
 }
+*/
 
 /* Should probably just get rid of this... the only code that cares is
  * part of the core anyway (and in fact, it isn't publicised to other
@@ -1680,6 +1686,7 @@
     return NULL;
 }
 
+/*
 static const char *satisfy(cmd_parms *cmd, void *c_, const char *arg)
 {
     core_dir_config *c = c_;
@@ -1704,6 +1711,7 @@
 
     return NULL;
 }
+*/
 
 /*
  * Report a missing-'>' syntax error.
@@ -3219,8 +3227,10 @@
   "specified URL paths"),
 AP_INIT_RAW_ARGS("<FilesMatch", filesection, (void*)1, OR_ALL,
   "Container for directives affecting files matching specified patterns"),
+/*
 AP_INIT_TAKE1("Satisfy", satisfy, NULL, OR_AUTHCFG,
   "access policy if both allow and require used ('all' or 'any')"),
+*/  
 #ifdef GPROF
 AP_INIT_TAKE1("GprofDir", set_gprof_dir, NULL, RSRC_CONF,
   "Directory to plop gmon.out files"),
@@ -3709,7 +3719,7 @@
 {
     logio_add_bytes_out = APR_RETRIEVE_OPTIONAL_FN(ap_logio_add_bytes_out);
     ident_lookup = APR_RETRIEVE_OPTIONAL_FN(ap_ident_lookup);
-    authz_ap_requires = APR_RETRIEVE_OPTIONAL_FN(authz_ap_requires);
+/*    authz_ap_requires = APR_RETRIEVE_OPTIONAL_FN(authz_ap_requires); */
     authz_ap_some_auth_required = APR_RETRIEVE_OPTIONAL_FN(authz_some_auth_required);
     authn_ap_auth_type = APR_RETRIEVE_OPTIONAL_FN(authn_ap_auth_type);
     authn_ap_auth_name = APR_RETRIEVE_OPTIONAL_FN(authn_ap_auth_name);



Mime
View raw message