commit e7e9597eaac03cbaa16c6ccbc01e35238da467ae
author Nathan Rini <nate@schedmd.com> Thu Aug 21 10:08:32 2025 -0600
committer Nathan Rini <nate@schedmd.com> Tue Sep 02 16:44:47 2025 -0600
tree f0ed488fc34ec71e59432a4b052c0b799a8de8b8
parent 16c3b5710779cb9e94a6c9b3fe89fcd87aef54da

slurmrestd - Convert _call_handler() to http_context_set_auth()

Avoid accessing args->context->auth directly.

Issue: 50189

src/slurmrestd/operations.c

diff --git a/src/slurmrestd/operations.c b/src/slurmrestd/operations.c
index 4dfb9d5..97a8f7b 100644
--- a/src/slurmrestd/operations.c
+++ b/src/slurmrestd/operations.c
@@ -46,6 +46,7 @@
 #include "src/common/xstring.h"
 #include "src/interfaces/serializer.h"
 
+#include "src/slurmrestd/http.h"
 #include "src/slurmrestd/operations.h"
 #include "src/slurmrestd/rest_auth.h"
 
@@ -500,22 +501,24 @@
 	data_t *resp = data_new();
 	char *body = NULL;
 	http_status_code_t e;
+	void *auth = NULL;
 
 	xassert(op_path);
 	debug3("%s: [%s] BEGIN: calling ctxt handler: 0x%"PRIXPTR"[%d] for path: %s",
 	       __func__, _name(args), (uintptr_t) op_path->callback,
 	       callback_tag, args->path);
 
+	auth = http_context_set_auth(args->context, NULL);
+
 	rc = wrap_openapi_ctxt_callback(_name(args), args->method, params,
-					query, callback_tag, resp,
-					args->context->auth, parser, op_path,
-					meta);
+					query, callback_tag, resp, auth, parser,
+					op_path, meta);
 
 	/*
 	 * Clear auth context after callback is complete. Client has to provide
 	 * full auth for every request already.
 	 */
-	FREE_NULL_REST_AUTH(args->context->auth);
+	FREE_NULL_REST_AUTH(auth);
 
 	if (data_get_type(resp) != DATA_TYPE_NULL) {
 		int rc2;