[🐛 | Bug]: Organization bootstrap fails

[Landeers]

Parsec version tested on:

v3.0.0-rc.8

Platforms tested on:

Windows

Bug description:

On custom server while bootstrap (join organization with bootstrap link), client received an BootstrapOrganizationErrorInternal error

Relevant output:

Server capture on debug level

[FirelightFlagboy]

Look if it's possible to inspect the header value Content-Type

[FirelightFlagboy]

The status 415 correspond to BadContentTypeOrInvalidBodyOrUnknownCommand

parsec-cloud/server/parsec/asgi/rpc.py

Line 255 in 1cc43a9

BadContentTypeOrInvalidBodyOrUnknownCommand = 415

[FirelightFlagboy]

It's returned by _handshake_abort_bad_content

That function is call during:

• _parse_auth_headers_or_abort if Content-Type header is application/msgpack

  parsec-cloud/server/parsec/asgi/rpc.py

  Lines 453 to 461 in 1cc43a9

  	parsed = _parse_auth_headers_or_abort(
  	headers=request.headers,
  	raw_organization_id=raw_organization_id,
  	with_authenticated_headers=False,
  	with_invited_headers=False,
  	with_sse_headers=False,
  	expected_accept_type=None,
  	expected_content_type=CONTENT_TYPE_MSGPACK,
  	)

  
  

  parsec-cloud/server/parsec/asgi/rpc.py

  Lines 339 to 340 in 1cc43a9

  	if expected_content_type and headers.get("Content-Type") != expected_content_type:
  	_handshake_abort_bad_content(api_version=settled_api_version)

• If during the parsing of the anonymous command the exception ValueError is raised

  parsec-cloud/server/parsec/asgi/rpc.py

  Lines 504 to 507 in 1cc43a9

  	try:
  	req = ANONYMOUS_CMDS_LOAD_FN[parsed.settled_api_version.version](body)
  	except ValueError:
  	_handshake_abort_bad_content(api_version=parsed.settled_api_version)

[FirelightFlagboy]

Look if it's possible to inspect the header value Content-Type

I've take a look about how to customize the access log provided by uvicorn

The issue encode/uvicorn#491 was a great help.

I was able to customize uvicorn with a custom access log format (see below)

diff --git a/server/parsec/asgi/__init__.py b/server/parsec/asgi/__init__.py
index 68ba698d3..66f6a2d42 100644
--- a/server/parsec/asgi/__init__.py
+++ b/server/parsec/asgi/__init__.py
@@ -1,6 +1,7 @@
 # Parsec Cloud (https://parsec.cloud) Copyright (c) BUSL-1.1 2016-present Scille SAS
 from __future__ import annotations
 
+import copy
 from pathlib import Path
 
 import uvicorn
@@ -8,6 +9,7 @@ from fastapi import FastAPI, Request
 from fastapi.staticfiles import StaticFiles
 from fastapi.templating import Jinja2Templates
 from starlette.types import Receive, Scope, Send
+import uvicorn.config
 
 from parsec._version import __version__ as parsec_version
 from parsec.asgi.administration import administration_router
@@ -93,6 +95,12 @@ async def serve_parsec_asgi_app(
         v_major, _ = parsec_version.split(".", 1)
         # ex: parsec/3
         server_header = f"parsec/{v_major}"
+    log_config = copy.deepcopy(uvicorn.config.LOGGING_CONFIG)
+    # Eg: "INFO 127.0.0.1 - "GET /anonymous/foobar HTTP/1.1" 200"
+    log_config["formatters"]["access"]["fmt"] = (
+        'foobar %(levelprefix)s %(client_addr)s - "%(request_line)s" %(status_code)s'
+    )
+
     # Note: Uvicorn comes with default values for incoming data size to
     # avoid DoS abuse, so just trust them on that ;-)
     config = uvicorn.Config(
@@ -107,6 +115,7 @@ async def serve_parsec_asgi_app(
         ssl_certfile=ssl_certfile,
         workers=workers,
         # TODO: configure access log format:
+        log_config=log_config,
         # Timestamp is added by the log processor configured in `parsec.logging`,
         # here we configure peer address + req line + rep status + rep body size + time
         # (e.g. "GET 88.0.12.52:54160 /foo 1.1 404 823o 12343ms")

Sadly, The log formatter don't have access to the request scope (HTTPScope) that contains the headers. Where are limited to client_addr, request_line and status_code (see uvicorn/logging.py::AccessFormatter)

[mmmarcos]

According to @Landeers this is no longer a blocking issue because a workaround was found for the reverse proxy redirects