Skip to content

Conversation

@defp
Copy link
Contributor

@defp defp commented Sep 26, 2025

alfter config CRUD_ADMIN_REDIS_SSL, will fail

  File "/Users/lidashuang/Projects/nocode/vision-rest/src/app/main.py", line 24, in <module>
    admin = create_admin_interface()
            ^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/lidashuang/Projects/nocode/vision-rest/src/app/admin/initialize.py", line 31, in create_admin_interface
    admin = CRUDAdmin(
            ^^^^^^^^^^
  File "/Users/lidashuang/Projects/nocode/vision-rest/.venv/lib/python3.11/site-packages/crudadmin/admin_interface/crud_admin.py", line 465, in __init__
    self._session_backend_kwargs = self._configure_session_backend(
                                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/lidashuang/Projects/nocode/vision-rest/.venv/lib/python3.11/site-packages/crudadmin/admin_interface/crud_admin.py", line 1343, in _configure_session_backend
    redis_config_obj = RedisConfig(**redis_config)
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/lidashuang/Projects/nocode/vision-rest/.venv/lib/python3.11/site-packages/pydantic/main.py", line 253, in __init__
    validated_self = self.__pydantic_validator__.validate_python(data, self_instance=self)
                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
pydantic_core._pydantic_core.ValidationError: 1 validation error for RedisConfig
ssl
  Extra inputs are not permitted [type=extra_forbidden, input_value=False, input_type=bool]
    For further information visit https://errors.pydantic.dev/2.11/v/extra_forbidden

@defp
Copy link
Contributor Author

defp commented Sep 26, 2025

The issue was caused by passing an unsupported ssl parameter to the CRUDAdmin's RedisConfig model, which resulted in a Pydantic validation error due to the model's strict field validation that doesn't allow extra inputs.

@igorbenav
Copy link
Collaborator

Awesome, thanks!

@igorbenav igorbenav merged commit 3f5ded5 into benavlabs:main Sep 26, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants