Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Arreglo generación del audio del captcha #581

Merged
merged 6 commits into from
Sep 25, 2023

Conversation

marcorichetta
Copy link
Contributor

@marcorichetta marcorichetta commented Sep 7, 2023

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:

Pruebas realizadas:

  • OS: Endeavour OS 6.4.12-arch1-1

  • Docker: 24.0.5

  • Docker Compose: 2.20.3

  • Rebuildeé el proyecto localmente con las dependencias de este PR

  • Pude descargar el audio captcha y reproducirlo (inglés)

  • Reproducir el audio desde la web

Cómo probar los cambios:

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

@facundobatista
Copy link
Member

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

Saludos!

@marcorichetta
Copy link
Contributor Author

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
Copy link
Contributor Author

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

@marcorichetta marcorichetta marked this pull request as ready for review September 9, 2023 21:14
Copy link
Member

@facundobatista facundobatista left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Buenísimo, gracias!

@facundobatista facundobatista merged commit 958cc9b into PyAr:master Sep 25, 2023
1 check passed
@facundobatista
Copy link
Member

Voy a deployar con esto y comento.

@facundobatista
Copy link
Member

Funciona 10 puntos! Gracias!

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.

Está rota la registración porque no funciona el captcha accesible CAPTCHA
2 participants