fix(v1.51): #1948 passing wrapper class to struct logging config causes pydantic exception

[JacobCoffee]

Pull Request Checklist

• New code has 100% test coverage
• (If applicable) The prose documentation has been updated to reflect the changes introduced by this PR
• (If applicable) The reference documentation has been updated to reflect the changes introduced by this PR

Description

• This PR is to fix Bug: passing wrapper_class to StructLoggingConfig causes Pydantic exception #1948, but the only way I have been able to resolve it is through changing the type hint to Any. I do not think this is the right resolution, i have tried do a couple of other things but nothing has worked... would like input so I can learn.

Close Issue(s)

• Closes Bug: passing wrapper_class to StructLoggingConfig causes Pydantic exception #1948

[JacobCoffee]

It is made about the non-method member _context, but I am not sure how we resolve this in 2.x besides dropping the BaseModel. I tried making a LocalBindableLogger, but that that lost _context and having wrapper_class inherit that
wrapper_class: Optional[Type[LocalBindableLogger]] = Field(default_factory=default_wrapper_class) felt wrong? Open to learn..

pydantic.error_wrappers.ValidationError: 1 validation error for StructLoggingConfig
wrapper_class
  Protocols with non-method members don't support issubclass() (type=type_error)

@runtime_checkable
class BindableLogger(Protocol):
    """
    **Protocol**: Methods shared among all bound loggers and that are relied on
    by *structlog*.

    .. versionadded:: 20.2
    """

    _context: Context

    def bind(self, **new_values: Any) -> BindableLogger:
        ...

    def unbind(self, *keys: str) -> BindableLogger:
        ...

    def try_unbind(self, *keys: str) -> BindableLogger:
        ...

    def new(self, **new_values: Any) -> BindableLogger:
        ...