aws / aws/aws-dynamodb-encryption-python

Wrong formatting in the deprecated python warning

Abierto
#524 3 comentarios 0 reacciones 0 asignados Ver en GitHub
good first issue maintenance
Lenguaje dominante
Python
Estrellas
95
Forks
57
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

### Security issue notifications

If you discover a potential security issue in the Amazon DynamoDB Encryption Client we ask that you notify AWS Security via our [vulnerability reporting page](https://aws.amazon.com/security/vulnerability-reporting/). Please do **not** create a public GitHub issue.

### Problem:
https://github.com/aws/aws-dynamodb-encryption-python/blob/35ee30aea5a93b4446677fdf1f14edc3ca3c0452/src/dynamodb_encryption_sdk/compatability.py#L39

The _minimum_version_ and _date_ seem to have been swapped in the deprecated python warning.
### Solution:

Putting the `params["date"]` before the `minimum_version[0]` should fix the issue.

### Out of scope:

Is there anything the solution will intentionally NOT address?

[//]: # (NOTE: If you believe this might be a security issue, please email aws-security@amazon.com instead of creating a GitHub issue. For more details, see the AWS Vulnerability Reporting Guide: https://aws.amazon.com/security/vulnerability-reporting/ )

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

Open src/dynamodb_encryption_sdk/compatability.py at line 39 and inspect the deprecated Python warning formatting. Verify which values are currently used for the minimum version and date, then confirm the warning presents them in the intended order described in the issue.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
python
Área
backend
Tipo de issue
Error
Dificultad
1/5
Tiempo estimado
Menos de una hora
Estado de actividad
Estancado
Claridad
Bien especificado
Aptitud para principiantes
35/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.