Vulnerabilities pareja3

iTzGooDLife commented 2 months ago

Descripción

Se agregan los avances de la sección Vulnerabilities. Actualmente están funcionales las siguientes acciones:

CRUD Vulnerabilities en multiples idiomas.
Implementación CVSS 3.1
Merge Vulnerabilities
Recomendación de CWE con filtrado bajo un % (arbitrario, abierto a cambios)
Filtrado de idioma mediante el SelectDropdown
Se agregó la dependencia ae-cvss-calculator (se actualizó el package.json)

Con estos cambios, la sección de Vulnerabilities estaría finalizada, dónde en un futuro se le debería hacer un único cambio al botón de "FindAudits", donde se debe integrar con la sección de audits. En caso de encontrar algún error, avisar a la brevedad para su corrección.

Como consideración, no tiene warning ni errores de linter y el único problema para buildear fue arreglado en main, en los screenshots se muestra el detalle de estos problemas para buildear.

Motivación y Contexto

Migración de frontend.

¿Cómo ha sido probado?

Para las pruebas se han agregado, editado, eliminado vulnerabilidades usando diferentes idiomas. Además, se han hecho merge de vulnerabilidades y recomendaciones de CWEs.

Capturas de pantalla (si es apropiado):

Tipos de cambios

[ ] Bugfix (cambio que no interrumpe el funcionamiento y que soluciona un problema)
[X] New feature (cambio que no interrumpe el funcionamiento y que añade funcionalidad)
[ ] Breaking change (corrección o funcionalidad que podría causar que la funcionalidad existente cambie)

Lista de verificación:

[X] Mi código sigue el estilo de código de este proyecto.
[ ] Mi cambio requiere una modificación en la documentación.
[ ] He actualizado la documentación en consecuencia.

jllanosg commented 2 months ago

Si el problema fue resuelto en main, no mergeaste main hacia tu rama?

iTzGooDLife commented 2 months ago

Si el problema fue resuelto en main, no mergeaste main hacia tu rama?

No lo noté hasta que se hizo el merge del último PR y son muchos cambios que no se debian incluir en este PR.

iTzGooDLife commented 2 months ago

Si el problema fue resuelto en main, no mergeaste main hacia tu rama?

No lo noté hasta que se hizo el merge del último PR y son muchos cambios que no se debian incluir en este PR.

Igual esto es normal, si hubieron 4h de diferenci entre esos cambios.

jllanosg commented 2 months ago

No entendí, pero por favor sincroniza tu rama con main para probar.

iTzGooDLife commented 2 months ago

No entendí, pero por favor sincroniza tu rama con main para probar.

No. Si se sincroniza con main, ahora se incluirán todos los cambios recientes que son más de 125 archivos. Te recuerdo que este merge es hacia la rama development, la cual no tiene esos cambios aún. Además, no veo el problema de mantenerlo como está, ya que al hacer el merge, se mantendrán esos cambios de main ya que son los más recientes.

jllanosg commented 2 months ago

No entendí, pero por favor sincroniza tu rama con main para probar.

No. Si se sincroniza con main, ahora se incluirán todos los cambios recientes que son más de 125 archivos. Te recuerdo que este merge es hacia la rama development, la cual no tiene esos cambios aún. Además, no veo el problema de mantenerlo como está, ya que al hacer el merge, se mantendrán esos cambios de main ya que son los más recientes.

En realidad hay que mergear main hacia development, luego deberás mergear development hacia tu rama (sincronizar). Podrías ver esto @caverav ?

iTzGooDLife commented 2 months ago

De igual manera son archivos independientes de los que se agregan en este PR, el resultado final será el mismo, pero si quieres hacer sincronizar de main a development está bien.

iTzGooDLife commented 2 months ago

Realizando los cambios mencionados de main buildea sin problemas.

caverav commented 2 months ago

No entendí, pero por favor sincroniza tu rama con main para probar.

No. Si se sincroniza con main, ahora se incluirán todos los cambios recientes que son más de 125 archivos. Te recuerdo que este merge es hacia la rama development, la cual no tiene esos cambios aún. Además, no veo el problema de mantenerlo como está, ya que al hacer el merge, se mantendrán esos cambios de main ya que son los más recientes.

En realidad hay que mergear main hacia development, luego deberás mergear development hacia tu rama (sincronizar). Podrías ver esto @caverav ?

Sincronicé development

iTzGooDLife commented 2 months ago

@iTzGooDLife el PR tiene un screenshot adjunto, pero no se explica contexto ni lo que se quiere comunicar con la imagen, porfa modifícalo o elimina esa imagen si no tiene utilidad

Si se explica el contexto y está al final de la descripción, aunque fue mala mia por no destacarlo o mencionarlo nuevamente

caverav commented 2 months ago

@iTzGooDLife el PR tiene un screenshot adjunto, pero no se explica contexto ni lo que se quiere comunicar con la imagen, porfa modifícalo o elimina esa imagen si no tiene utilidad

Si se explica el contexto y está al final de la descripción, aunque fue mala mia por no destacarlo o mencionarlo nuevamente

Me di cuenta, por eso eliminé el comentario

iTzGooDLife commented 2 months ago

El placeholder del título en la creación de una vuln es "search"

Aahh sii, vale por la acotación. Diria que ignoren por mientras los placeholders. En un rato los arreglo y agregaré algunas cosas al i18.

iTzGooDLife commented 2 months ago

El placeholder del título en la creación de una vuln es "search"

Ya deberian estar corregidos los placeholders y además, revisar nuevamente por favor.

Icksir commented 2 months ago

Es normal que al intentar mergear vulnerabilidades de dos idiomas, desaparezca la opción de seleccionarlas al seleccionar el segundo idioma? O no entiendo cómo ocupar bien esta funcionalidad

Selección de vulnerabilidad en idioma en
Selección de idioma es (desaparece vulnerabilidad en idioma en)
Al apretar merge, produce error

iTzGooDLife commented 2 months ago

Es normal que al intentar mergear vulnerabilidades de dos idiomas, desaparezca la opción de seleccionarlas al seleccionar el segundo idioma? O no entiendo cómo ocupar bien esta funcionalidad

1. Selección de vulnerabilidad en idioma **en**
   ![image](https://private-user-images.githubusercontent.com/66755477/364430818-298f1e2f-b507-4e9f-8de0-09df3583381c.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjU0NjIxNjMsIm5iZiI6MTcyNTQ2MTg2MywicGF0aCI6Ii82Njc1NTQ3Ny8zNjQ0MzA4MTgtMjk4ZjFlMmYtYjUwNy00ZTlmLThkZTAtMDlkZjM1ODMzODFjLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA5MDQlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwOTA0VDE0NTc0M1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTY4NzU5MGVkMzIyMmFhMDVmODM4ZmFjM2EzNWUxZDA5OGUxOGJiYjkyZDc4YTg4Yzk2MzVjMjJjNjk1YWIyNTgmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.kxdzdnW2vYi94zbXcMbAPn8ZBHnXxnRxnvHm7VZMhuU)

2. Selección de idioma es (desaparece vulnerabilidad en idioma **en**)
   ![image](https://private-user-images.githubusercontent.com/66755477/364431233-39c970cf-560e-4158-872d-f20c54c52f9c.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjU0NjIxNjMsIm5iZiI6MTcyNTQ2MTg2MywicGF0aCI6Ii82Njc1NTQ3Ny8zNjQ0MzEyMzMtMzljOTcwY2YtNTYwZS00MTU4LTg3MmQtZjIwYzU0YzUyZjljLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA5MDQlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwOTA0VDE0NTc0M1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTFlMzE4ODA4ZDc1MzllYjcwZmQ1MWU3NjQyNDg4ZGMxYTdiNmY3MDEzNjgwM2Q5ZmU0MTJmMTRiZjlkNDhlYTImWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.sUuu8RJkFCMHlHzdFu_uaxAlh54RpuQ3mTQMH88eXQA)

3. Al apretar merge, produce error
   ![image](https://private-user-images.githubusercontent.com/66755477/364431402-4453c966-dc61-48d4-9bbb-087345db8090.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjU0NjIxNjMsIm5iZiI6MTcyNTQ2MTg2MywicGF0aCI6Ii82Njc1NTQ3Ny8zNjQ0MzE0MDItNDQ1M2M5NjYtZGM2MS00OGQ0LTliYmItMDg3MzQ1ZGI4MDkwLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA5MDQlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwOTA0VDE0NTc0M1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTg2YWE4Y2NhMzVlYzMwYTNiYzUzZjhiOGRkN2ZkNGZjNjQxMzczM2MzNTUyMDJlNjMyMzIzYWMwY2MyNmVkNGUmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.nbrMwlSwTMcl9xqrLtoJwyRhLRRst0Qwg3S5K-XVSVE)

Si entendí bien, debe ser normal si es que la vulnerabilidad seleccionada ya tiene cambios en ambos idiomas. Explicaré como funciona: Por ejemplo, si tengo una vulnerabilidad en español (vuln A) y tengo otra en inglés (vuln B), estas se pueden mergear y sería solo una con idioma español e inglés. Luego, esta vulnerabilidad ya no aparecerá como opción al intentar mergear de español con inglés pq ya posee ambos idiomas.

Además, para que funcione el merge, deben haber dos vulnerabilidades seleccionadas (una a cada lado)

Es un poco complicado el concepto, recomiendo leer la documentación para más detalles: https://pwndoc.github.io/pwndoc/#/vulnerabilities?id=merge

Icksir commented 2 months ago

Es normal que al intentar mergear vulnerabilidades de dos idiomas, desaparezca la opción de seleccionarlas al seleccionar el segundo idioma? O no entiendo cómo ocupar bien esta funcionalidad

1. Selección de vulnerabilidad en idioma **en**
   ![image](https://private-user-images.githubusercontent.com/66755477/364430818-298f1e2f-b507-4e9f-8de0-09df3583381c.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjU0NjIxNjMsIm5iZiI6MTcyNTQ2MTg2MywicGF0aCI6Ii82Njc1NTQ3Ny8zNjQ0MzA4MTgtMjk4ZjFlMmYtYjUwNy00ZTlmLThkZTAtMDlkZjM1ODMzODFjLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA5MDQlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwOTA0VDE0NTc0M1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTY4NzU5MGVkMzIyMmFhMDVmODM4ZmFjM2EzNWUxZDA5OGUxOGJiYjkyZDc4YTg4Yzk2MzVjMjJjNjk1YWIyNTgmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.kxdzdnW2vYi94zbXcMbAPn8ZBHnXxnRxnvHm7VZMhuU)

2. Selección de idioma es (desaparece vulnerabilidad en idioma **en**)
   ![image](https://private-user-images.githubusercontent.com/66755477/364431233-39c970cf-560e-4158-872d-f20c54c52f9c.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjU0NjIxNjMsIm5iZiI6MTcyNTQ2MTg2MywicGF0aCI6Ii82Njc1NTQ3Ny8zNjQ0MzEyMzMtMzljOTcwY2YtNTYwZS00MTU4LTg3MmQtZjIwYzU0YzUyZjljLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA5MDQlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwOTA0VDE0NTc0M1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTFlMzE4ODA4ZDc1MzllYjcwZmQ1MWU3NjQyNDg4ZGMxYTdiNmY3MDEzNjgwM2Q5ZmU0MTJmMTRiZjlkNDhlYTImWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.sUuu8RJkFCMHlHzdFu_uaxAlh54RpuQ3mTQMH88eXQA)

3. Al apretar merge, produce error
   ![image](https://private-user-images.githubusercontent.com/66755477/364431402-4453c966-dc61-48d4-9bbb-087345db8090.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjU0NjIxNjMsIm5iZiI6MTcyNTQ2MTg2MywicGF0aCI6Ii82Njc1NTQ3Ny8zNjQ0MzE0MDItNDQ1M2M5NjYtZGM2MS00OGQ0LTliYmItMDg3MzQ1ZGI4MDkwLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA5MDQlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwOTA0VDE0NTc0M1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTg2YWE4Y2NhMzVlYzMwYTNiYzUzZjhiOGRkN2ZkNGZjNjQxMzczM2MzNTUyMDJlNjMyMzIzYWMwY2MyNmVkNGUmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.nbrMwlSwTMcl9xqrLtoJwyRhLRRst0Qwg3S5K-XVSVE)

Si entendí bien, debe ser normal si es que la vulnerabilidad seleccionada ya tiene cambios en ambos idiomas. Explicaré como funciona: Por ejemplo, si tengo una vulnerabilidad en español (vuln A) y tengo otra en inglés (vuln B), estas se pueden mergear y sería solo una con idioma español e inglés. Luego, esta vulnerabilidad ya no aparecerá como opción al intentar mergear de español con inglés pq ya posee ambos idiomas.

Además, para que funcione el merge, deben haber dos vulnerabilidades seleccionadas (una a cada lado)

Es un poco complicado el concepto, recomiendo leer la documentación para más detalles: https://pwndoc.github.io/pwndoc/#/vulnerabilities?id=merge

Ahí me funcionó, lo que no me queda claro es cómo hacer funcionar el modelo recomendador de CWE

jllanosg commented 2 months ago

Voy a echarle una última ojeada y si no encuentro nada voy a mergear. @caverav dejaste changes requested pero no cambiaste el status cuando fueron resueltos, voy a hacer dismiss de ello.

caverav / auditforge