From 715dbd5206b1f0eba67a19331f259fdb3ee9886c Mon Sep 17 00:00:00 2001 From: rgaudin Date: Thu, 6 Feb 2025 12:46:53 +0000 Subject: [PATCH] Replaced HTTP redirect on / to HTML redirect Apple Pay Verification system seems unhappy with redirects on / when it is verifying domain. This ought to keep it happy while maintaining the feature for end users --- donation-api/src/donation_api/entrypoint.py | 25 +++++++++++++++++---- 1 file changed, 21 insertions(+), 4 deletions(-) diff --git a/donation-api/src/donation_api/entrypoint.py b/donation-api/src/donation_api/entrypoint.py index 2c6d95bf..77c294ce 100644 --- a/donation-api/src/donation_api/entrypoint.py +++ b/donation-api/src/donation_api/entrypoint.py @@ -1,8 +1,8 @@ from http import HTTPStatus -from fastapi import FastAPI +from fastapi import FastAPI, Response from fastapi.middleware.cors import CORSMiddleware -from fastapi.responses import PlainTextResponse, RedirectResponse +from fastapi.responses import PlainTextResponse from donation_api import stripe from donation_api.__about__ import __description__, __title__, __version__ @@ -19,9 +19,26 @@ def create_app() -> FastAPI: ) @app.get("/") + @app.head("/") async def _(): - """Redirect to root of latest version of the API""" - return RedirectResponse(f"{PREFIX}/", status_code=HTTPStatus.PERMANENT_REDIRECT) + """HTML Redirect to root of latest version of the API + + Purposedly not an HTTP redirect as Apple Pay verification system + doesnt like it (apparently)""" + + return Response( + f""" + + + + +Donation API + + +

Current version is located at {PREFIX}/

+ +""" + ) # could be done on infra ; this is a handy shortcut if conf.merchantid_domain_association: