Arreglo generación del audio del captcha

marcorichetta commented 1 year ago

Cambios propuestos:

Actualizar dependencia django-simple-captcha
Instalar flite para generación de audio del captcha
Definir DEFAULT_AUTO_FIELD para evitar warnings de Django >= 3.2 Source

WARNINGS:
models.Model: (models.W042) Auto-created primary key used when not defining a primary key type, by default 'django.db.models.AutoField'.
    HINT: Configure the DEFAULT_AUTO_FIELD setting or the ...

Temas tratados:

Closes #479
Closes #579

Pruebas realizadas:

OS: Endeavour OS 6.4.12-arch1-1
Docker: 24.0.5
Docker Compose: 2.20.3
[x] Rebuildeé el proyecto localmente con las dependencias de este PR
[x] Pude descargar el audio captcha y reproducirlo (inglés)
[x] Reproducir el audio desde la web

Cómo probar los cambios:

git pull de esta rama
rebuildear imagen de docker o make start
http://localhost:8000

facundobatista commented 1 year ago

Hola! Gracias por este laburo! Pregunta, ¿ya estaría para revisar y aprobar este PR? (porque dice "draft"...)

Saludos!

marcorichetta commented 1 year ago

Hola Facu. Me olvidé de agregar en la descripción que no logré reproducir el audio desde la web, como se podía antes.

El problema es que no se está usando el template custom definido acá https://github.com/marcorichetta/pyarweb/blob/09b7b629b6bfdbd39b0d4895dc0c8de440bec360/pyarweb/settings/base.py#L194

Me parece que hasta no poder reproducir el audio no agrega mucho este PR.

marcorichetta commented 1 year ago

Finalmente pude reproducir el audio desde el browser. También modifiqué el form para que sirva un captcha con mejor resolución.

facundobatista commented 1 year ago

Voy a deployar con esto y comento.

facundobatista commented 1 year ago

Funciona 10 puntos! Gracias!

PyAr / pyarweb