Include ASGI scope root_path in url used to validate mAuth signature

[pburke-mdsol]

This fixes an issue encountered with a FastAPI setup similar to the the example provided below. When using a sub application mounted into a parent application the value of scope["path"] that the middleware attached to the sub application sees is the path without the prefix the application was mounted at. The route prefix is stored in scope["root_path"]. Both need to be combined when creating the URL used for mAuth signature verification.

For example using the application below, for a request to "/subapp/with_auth": scope["path"] will be "/with_auth" and scope["root_path"] will be "/subapp".

from fastapi import FastAPI
from mauth_client.middlewares import MAuthASGIMiddleware

app = FastAPI()


@app.get("/no_auth")
async def no_auth():
    return {"message": "no auth"}


subapp = FastAPI()
subapp.add_middleware(MAuthASGIMiddleware)


@subapp.get("/with_auth")
async def with_auth():
    return {"message": "with auth"}


app.mount("/subapp", subapp)

[ejinotti-mdsol]

Looks good, thx for contributing. Just needs: version bump, changelog, and a test case 👍.

[pburke-mdsol]

Looks good, thx for contributing. Just needs: version bump, changelog, and a test case 👍.

Should I bump the version to 1.6.2?

[ejinotti-mdsol]

yea, 1.6.2 seems correct. patch version for bug fixes: https://semver.org/

[pburke-mdsol]

OK, should hopefully be ready for re-review.

[ejinotti-mdsol]

created new tag/release @pburke-mdsol

[pburke-mdsol]

created new tag/release @pburke-mdsol

Thanks!