Skip to content

Feat/sec 15 register jwt report#298

Merged
ericmt-98 merged 3 commits into
ericmt-98:mainfrom
larryjay007:feat/sec-15-register-jwt-report
Jul 6, 2026
Merged

Feat/sec 15 register jwt report#298
ericmt-98 merged 3 commits into
ericmt-98:mainfrom
larryjay007:feat/sec-15-register-jwt-report

Conversation

@larryjay007

Copy link
Copy Markdown
Contributor

Description

Reporte de seguridad SEC-15: /users/register emite JWT para dirección Stellar no verificada (bypass de firma).

Related Issue

Closes #246

Type of Change

  • Documentation update

Checklist

  • Code follows project style guidelines
  • Self-review completed
  • Documentation updated
  • No new linting issues
  • Changes verified locally

@drips-wave

drips-wave Bot commented Jun 30, 2026

Copy link
Copy Markdown

@larryjay007 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@larryjay007

Copy link
Copy Markdown
Contributor Author

please review, thanks for letting me contribute in your project buddy.

@larryjay007

Copy link
Copy Markdown
Contributor Author

HI @ericmt-98 , could you please look into this. thanks

@ericmt-98

Copy link
Copy Markdown
Owner

¡Gracias por el reporte de SEC-15, es válido y está bien documentado! 🙌

Antes de mergear: este PR también incluye docs/security-reports/SEC-23-token-keys-inconsistentes.md, pero ese reporte ya fue aceptado y está mergeado en main desde otro PR, con contenido idéntico. ¿Puedes quitar ese archivo de esta rama (dejando solo SEC-15-register-jwt-sin-firma.md)? Así evitamos duplicar crédito por un hallazgo que ya se cerró.

Con ese ajuste, este PR queda listo para mergear.

@larryjay007

Copy link
Copy Markdown
Contributor Author

¡Gracias por la revisión!
He eliminado el archivo duplicado SEC-23-token-keys-inconsistentes.md (ya estaba mergeado en otro PR).
El PR ahora solo contiene el reporte SEC-15.
Listo para mergear cuando quieras.

@larryjay007

Copy link
Copy Markdown
Contributor Author

Hey Buddy @ericmt-98 , I want to remind to merge and close the issue.

@ericmt-98 ericmt-98 merged commit 320611c into ericmt-98:main Jul 6, 2026
2 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.

[SEC-15] /users/register emite JWT para dirección Stellar no verificada (bypass de firma)

2 participants