summaryrefslogtreecommitdiffstats
path: root/server
diff options
context:
space:
mode:
authorSimo Sorce <ssorce@redhat.com>2009-05-15 18:48:57 -0400
committerSimo Sorce <ssorce@redhat.com>2009-05-18 16:08:24 -0400
commitcfd69419bece0047fd373c64e56e5eaf2a4c89a9 (patch)
tree134457e744cef51a046b38a66e66866e5f9a9412 /server
parent66c727e0e7b34d19cdb8dbdc0a0fae15d9d5ff25 (diff)
downloadsssd-cfd69419bece0047fd373c64e56e5eaf2a4c89a9.tar.gz
sssd-cfd69419bece0047fd373c64e56e5eaf2a4c89a9.tar.xz
sssd-cfd69419bece0047fd373c64e56e5eaf2a4c89a9.zip
Implement approximate offline detection in proxy
This will blackout any request to the backend for 15 seconds, then will allow again to retry.
Diffstat (limited to 'server')
-rw-r--r--server/providers/proxy.c103
1 files changed, 98 insertions, 5 deletions
diff --git a/server/providers/proxy.c b/server/providers/proxy.c
index 175670ada..c6ab4e340 100644
--- a/server/providers/proxy.c
+++ b/server/providers/proxy.c
@@ -59,6 +59,7 @@ struct proxy_nss_ops {
struct proxy_ctx {
struct proxy_nss_ops ops;
+ bool offline;
};
struct proxy_auth_ctx {
@@ -76,6 +77,49 @@ static void cache_password(struct be_req *req,
static void proxy_reply(struct be_req *req,
int error, const char *errstr);
+static void offline_timeout(struct tevent_context *ev, struct tevent_timer *tt,
+ struct timeval tv, void *pvt)
+{
+ struct proxy_ctx *ctx;
+
+ ctx = talloc_get_type(pvt, struct proxy_ctx);
+ ctx->offline = false;
+}
+
+static void go_offline(struct be_ctx *be_ctx)
+{
+ struct proxy_ctx *ctx;
+ struct tevent_timer *tt;
+ struct timeval timeout;
+ int ret;
+
+ ctx = talloc_get_type(be_ctx->pvt_id_data, struct proxy_ctx);
+
+ ret = gettimeofday(&timeout, NULL);
+ if (ret == -1) {
+ DEBUG(1, ("gettimeofday failed [%d][%s].\n", errno, strerror(errno)));
+ return;
+ }
+ timeout.tv_sec += 15; /* TODO: get from conf */
+
+ tt = tevent_add_timer(be_ctx->ev, ctx, timeout, offline_timeout, ctx);
+ if (tt == NULL) {
+ DEBUG(1, ("Failed to add timer\n"));
+ return;
+ }
+
+ ctx->offline = true;
+}
+
+static bool is_offline(struct be_ctx *be_ctx)
+{
+ struct proxy_ctx *ctx;
+
+ ctx = talloc_get_type(be_ctx->pvt_id_data, struct proxy_ctx);
+
+ return ctx->offline;
+}
+
static int proxy_internal_conv(int num_msg, const struct pam_message **msgm,
struct pam_response **response,
void *appdata_ptr) {
@@ -436,10 +480,15 @@ static void get_pw_name(struct be_req *req, char *name)
ret = sysdb_transaction(data, req->be_ctx->sysdb, set_pw_name, data);
break;
+ case NSS_STATUS_UNAVAIL:
+ /* "remote" backend unavailable. Enter offline mode */
+ DEBUG(2, ("proxy returned UNAVAIL error, going offline!\n"));
+ go_offline(req->be_ctx);
+ return proxy_reply(req, EAGAIN, "Offline");
+
default:
- DEBUG(2, ("proxy -> getpwnam_r failed for '%s' (%d)[%s]\n",
- name, ret, strerror(ret)));
- return proxy_reply(req, ret, "Operation failed");
+ DEBUG(2, ("proxy -> getpwnam_r failed for '%s' <%d>\n", name, status));
+ return proxy_reply(req, EOK, "Operation failed");
}
if (ret != EOK) {
@@ -496,6 +545,12 @@ static void get_pw_uid(struct be_req *req, uid_t uid)
ret = sysdb_transaction(data, req->be_ctx->sysdb, set_pw_name, data);
break;
+ case NSS_STATUS_UNAVAIL:
+ /* "remote" backend unavailable. Enter offline mode */
+ DEBUG(2, ("proxy returned UNAVAIL error, going offline!\n"));
+ go_offline(req->be_ctx);
+ return proxy_reply(req, EAGAIN, "Offline");
+
default:
DEBUG(2, ("proxy -> getpwuid_r failed for '%lu' (%d)[%s]\n",
(unsigned long)uid, ret, strerror(ret)));
@@ -582,6 +637,12 @@ retry:
}
break;
+ case NSS_STATUS_UNAVAIL:
+ /* "remote" backend unavailable. Enter offline mode */
+ DEBUG(2, ("proxy returned UNAVAIL error, going offline!\n"));
+ go_offline(data->req->be_ctx);
+ return proxy_return(data, EAGAIN, NULL);
+
default:
DEBUG(2, ("proxy -> getpwent_r failed (%d)[%s]\n",
ret, strerror(ret)));
@@ -709,6 +770,12 @@ static void get_gr_name(struct be_req *req, char *name)
ret = sysdb_transaction(data, req->be_ctx->sysdb, set_gr_name, data);
break;
+ case NSS_STATUS_UNAVAIL:
+ /* "remote" backend unavailable. Enter offline mode */
+ DEBUG(2, ("proxy returned UNAVAIL error, going offline!\n"));
+ go_offline(req->be_ctx);
+ return proxy_reply(req, EAGAIN, "Offline");
+
default:
DEBUG(2, ("proxy -> getgrnam_r failed for '%s' (%d)[%s]\n",
name, ret, strerror(ret)));
@@ -769,6 +836,12 @@ static void get_gr_gid(struct be_req *req, gid_t gid)
ret = sysdb_transaction(data, req->be_ctx->sysdb, set_gr_name, data);
break;
+ case NSS_STATUS_UNAVAIL:
+ /* "remote" backend unavailable. Enter offline mode */
+ DEBUG(2, ("proxy returned UNAVAIL error, going offline!\n"));
+ go_offline(req->be_ctx);
+ return proxy_reply(req, EAGAIN, "Offline");
+
default:
DEBUG(2, ("proxy -> getgrgid_r failed for '%lu' (%d)[%s]\n",
(unsigned long)gid, ret, strerror(ret)));
@@ -847,6 +920,12 @@ retry:
}
break;
+ case NSS_STATUS_UNAVAIL:
+ /* "remote" backend unavailable. Enter offline mode */
+ DEBUG(2, ("proxy returned UNAVAIL error, going offline!\n"));
+ go_offline(data->req->be_ctx);
+ return proxy_return(data, EAGAIN, NULL);
+
default:
DEBUG(2, ("proxy -> getgrent_r failed (%d)[%s]\n",
ret, strerror(ret)));
@@ -1093,6 +1172,12 @@ static void get_initgr_user(struct be_req *req, char *name)
break;
}
+ case NSS_STATUS_UNAVAIL:
+ /* "remote" backend unavailable. Enter offline mode */
+ DEBUG(2, ("proxy returned UNAVAIL error, going offline!\n"));
+ go_offline(req->be_ctx);
+ return proxy_reply(req, EAGAIN, "Offline");
+
default:
DEBUG(2, ("proxy -> getpwnam_r failed for '%s' (%d)[%s]\n",
name, ret, strerror(ret)));
@@ -1113,7 +1198,11 @@ static void proxy_check_online(struct be_req *req)
oreq = talloc_get_type(req->req_data, struct be_online_req);
- oreq->online = MOD_ONLINE;
+ if (is_offline(req->be_ctx)) {
+ oreq->online = MOD_OFFLINE;
+ } else {
+ oreq->online = MOD_ONLINE;
+ }
req->fn(req, EOK, NULL);
}
@@ -1127,6 +1216,10 @@ static void proxy_get_account_info(struct be_req *req)
ar = talloc_get_type(req->req_data, struct be_acct_req);
+ if (is_offline(req->be_ctx)) {
+ return proxy_reply(req, EAGAIN, "Offline");
+ }
+
switch (ar->entry_type) {
case BE_REQ_USER: /* user */
switch (ar->filter_type) {
@@ -1269,7 +1362,7 @@ int sssm_proxy_init(struct be_ctx *bectx,
void *handle;
int ret;
- ctx = talloc(bectx, struct proxy_ctx);
+ ctx = talloc_zero(bectx, struct proxy_ctx);
if (!ctx) {
return ENOMEM;
}