Skip to content

Conversation

@Shreyas-Madake
Copy link
Contributor

Updated the outdated reference URL in all ASVS 5.0 language versions.

The previous link:
https://owasp.org/www-community/Application_Threat_Modeling

has been replaced with the correct link:
https://owasp.org/www-project-threat-model/

Updated files:

  • 5.0/en/0x93-Appendix-D_Recommendations.md
  • 5.0/tr/0x93-Appendix-D_Recommendations.md
  • 5.0/ru/0x93-Appendix-D_Recommendations.md
  • 5.0/fr/0x93-Appendix-D_Recommendations.md

Fixes #3287

@elarlang
Copy link
Collaborator

elarlang commented Nov 12, 2025

I took a quick view of those projects, and I'm not convinced why "the new one" is better than "the old one"?

From a wider point of view, are those projects duplicates, or is the second one effectively a new version of the previous one? In both cases, those projects should be merged then, and the redirection done on the web page side, instead of fixing the link for every usage on the Internet (which is not realistic anyway).

Based on repo, "the new" project seems also a bit outdated?

The second question I have is, should we actually fix it in translations, as translation must be exactly the released version as it was (even in the case the released version contains a mistake). Otherwise, we have translations of bleeding-edge version that has the number of the released version, and that is not correct.

@Shreyas-Madake
Copy link
Contributor Author

I took a quick view of those projects, and I'm not convinced why "the new one" is better than "the old one"?

From a wider point of view, are those projects duplicates, or is the second one effectively a new version of the previous one? In both cases, those projects should be merged then, and the redirection done on the web page side, instead of fixing the link for every usage on the Internet (which is not realistic anyway).

Based on repo, "the new" project seems also a bit outdated?

The second question I have is, should we actually fix it in translations, as translation must be exactly the released version as it was (even in the case the released version contains a mistake). Otherwise, we have translations of bleeding-edge version that has the number of the released version, and that is not correct.

Thanks for the review.
If the two pages are the same or one replaces the other, they should be merged and the old URL should redirect on the OWASP site.
If the “new” page is outdated, we should pick one official page, update it, and redirect to it.

For translations: we must match the released English exactly. We shouldn’t change links only in translations. If English is updated, I’ll update the translation to match.

@Shreyas-Madake
Copy link
Contributor Author

Updated the outdated reference URL in all ASVS 5.0 language versions.

The previous link: https://owasp.org/www-community/Application_Threat_Modeling

has been replaced with the correct link: https://owasp.org/www-project-threat-model/

Updated files:

  • 5.0/en/0x93-Appendix-D_Recommendations.md
  • 5.0/tr/0x93-Appendix-D_Recommendations.md
  • 5.0/ru/0x93-Appendix-D_Recommendations.md
  • 5.0/fr/0x93-Appendix-D_Recommendations.md

Fixes #3287

Reverted tr/ru/fr to released content;
kept English updated. Ready for review.

@adamshostack
Copy link

I took a quick view of those projects, and I'm not convinced why "the new one" is better than "the old one"?

From a wider point of view, are those projects duplicates, or is the second one effectively a new version of the previous one? In both cases, those projects should be merged then, and the redirection done on the web page side, instead of fixing the link for every usage on the Internet (which is not realistic anyway).

One is a community page, the other is a project page. That's visible in the URL. We'll likely fix the underlying problem, but in the meantime I've asked that OWASP, in its official projects, link to other projects, rather than community pages.

@tghosth tghosth merged commit f6b7a63 into OWASP:master Nov 12, 2025
8 checks passed
@Shreyas-Madake Shreyas-Madake deleted the fix-threat-model-link branch November 13, 2025 03:32
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.

Fix threat modeling link

4 participants