Fix HTTP integration tests under parallel ctest execution.

Give each HTTP fixture its own macaroons secret, move httpparserlegacy off
port 7096 to avoid clashing with cache, and harden teardown cleanup.

Assisted-by: Cursor:Composer-2.5 CursorAI
Co-authored-by: Cursor <cursoragent@cursor.com>

Author: apeters1971

src/XrdHttp/README.md

@@ -205,7 +205,7 @@ ctest -R 'XRootD::(http|httpparser|httph2)' --output-on-failure
 |------------|--------|------|----------|
 | `XRootD::http` | `tests/XRootD/http.sh` | 7094 | Full HTTP/1.1 integration |
 | `XRootD::httpparser` | `tests/XRootD/httpparser.sh` | 7095 | llhttp header parsing |
-| `XRootD::httpparserlegacy` | `tests/XRootD/httpparserlegacy.sh` | 7096 | Legacy line parser regression |
+| `XRootD::httpparserlegacy` | `tests/XRootD/httpparserlegacy.sh` | 7098 | Legacy line parser regression |
 | `XRootD::httph2` | `tests/XRootD/httph2.sh` | 7097 | HTTPS + ALPN h2 (GET/PUT/HEAD/DELETE, same-connection reuse) |
 
 ## Source layout

src/XrdHttp/wire/README.md

@@ -36,6 +36,6 @@ cmake .. -DENABLE_HTTP=ON
 ## Tests
 
 - `tests/XRootD/httpparser` — llhttp smoke tests (port 7095)
-- `tests/XRootD/httpparserlegacy` — legacy parser regression (port 7096)
+- `tests/XRootD/httpparserlegacy` — legacy parser regression (port 7098)
 - `tests/XRootD/http` — full HTTP/1.1 integration suite (port 7094)
 - `tests/XRootD/httph2` — HTTPS + ALPN h2 smoke tests (port 7097, TLS fixture)

tests/XRootD/http.cfg

@@ -19,7 +19,7 @@ ofs.authlib libXrdMacaroons.so
 http.header2cgi Authorization authz strip-on-redirect
 http.exthandler xrdtpc libXrdHttpTPC.so
 http.exthandler xrdmacaroons libXrdMacaroons.so
-macaroons.secretkey $pwd/macaroons-secret
+macaroons.secretkey $pwd/$name/macaroons-secret
 
 xrootd.mongstream http use flush 3s send json 127.0.0.1:8888
 

tests/XRootD/http.sh

@@ -4,7 +4,7 @@ export XRD_PLUGINCONFDIR="${BINARY_DIR}/config"
 
 function setup_http() {
 	require_commands openssl curl
-	openssl rand -base64 -out macaroons-secret 64
+	openssl rand -base64 -out "${NAME}/macaroons-secret" 64
 	mkdir -p "${XRD_PLUGINCONFDIR}"
 	cat >| "${XRD_PLUGINCONFDIR}/http.conf" <<-EOF
 	url = http://*;https://*;dav://*;davs://*
@@ -14,9 +14,9 @@ function setup_http() {
 }
 
 function teardown_http() {
-	rm "${XRD_PLUGINCONFDIR}/http.conf"
-	rmdir "${XRD_PLUGINCONFDIR}"
-	rm macaroons-secret
+	rm -f "${XRD_PLUGINCONFDIR}/http.conf"
+	rmdir "${XRD_PLUGINCONFDIR}" 2>/dev/null || true
+	rm -f "${NAME}/macaroons-secret"
 }
 
 function test_http() {

tests/XRootD/httpparser.cfg

@@ -15,7 +15,7 @@ ofs.authlib libXrdMacaroons.so
 http.header2cgi Authorization authz strip-on-redirect
 http.exthandler xrdtpc libXrdHttpTPC.so
 http.exthandler xrdmacaroons libXrdMacaroons.so
-macaroons.secretkey $pwd/macaroons-secret
+macaroons.secretkey $pwd/$name/macaroons-secret
 
 http.staticheader -verb=OPTIONS Access-Control-Allow-Origin *
 http.staticheader Test 1

tests/XRootD/httpparser.sh

@@ -2,11 +2,11 @@
 
 function setup_httpparser() {
 	require_commands openssl curl
-	openssl rand -base64 -out macaroons-secret 64
+	openssl rand -base64 -out "${NAME}/macaroons-secret" 64
 }
 
 function teardown_httpparser() {
-	rm -f macaroons-secret
+	rm -f "${NAME}/macaroons-secret"
 }
 
 function test_httpparser() {

tests/XRootD/httpparserlegacy.cfg

@@ -1,5 +1,5 @@
 set name = httpparserlegacy
-set port = 7096
+set port = 7098
 
 set pwd = $PWD
 set src = $SOURCE_DIR
@@ -15,6 +15,6 @@ ofs.authlib libXrdMacaroons.so
 http.header2cgi Authorization authz strip-on-redirect
 http.exthandler xrdtpc libXrdHttpTPC.so
 http.exthandler xrdmacaroons libXrdMacaroons.so
-macaroons.secretkey $pwd/macaroons-secret
+macaroons.secretkey $pwd/$name/macaroons-secret
 
 continue $src/common.cfg

tests/XRootD/httpparserlegacy.sh

@@ -2,11 +2,11 @@
 
 function setup_httpparserlegacy() {
 	require_commands openssl curl
-	openssl rand -base64 -out macaroons-secret 64
+	openssl rand -base64 -out "${NAME}/macaroons-secret" 64
 }
 
 function teardown_httpparserlegacy() {
-	rm -f macaroons-secret
+	rm -f "${NAME}/macaroons-secret"
 }
 
 function test_httpparserlegacy() {