wstg icon indicating copy to clipboard operation
wstg copied to clipboard

Re-establish content with regard to testing for race conditions

Open martinbydefault opened this issue 4 years ago • 16 comments

I was trying to map Testing for Race Conditions (OWASP-AT-010) from OTGv3 ( https://wiki.owasp.org/index.php/Testing_for_Race_Conditions_(OWASP-AT-010) ) to the latest version but I can't find a section talking about race conditions.

The only one I found is WSTG-INFO-07 where it mentions this: Race - tests multiple concurrent instances of the application manipulating the same data.

But it lacks most of the content that was previously in OTGv3.

martinbydefault avatar Sep 24 '20 19:09 martinbydefault

https://github.com/OWASP/wstg/blob/master/document/4-Web_Application_Security_Testing/06-Session_Management_Testing/08-Testing_for_Session_Puzzling.md

Looks like it was removed in 4.0: https://wiki.owasp.org/index.php/OWASP_Testing_Guide_v4_Table_of_Contents

kingthorin avatar Sep 24 '20 19:09 kingthorin

Yes I saw that 4.0 doesn't have it. I don't get why this was removed, given that this is a very specific issue and there are even new techniques to exploit this, for example using BURP Suite Turbo Intruder plugin (https://portswigger.net/research/turbo-intruder-embracing-the-billion-request-attack)

martinbydefault avatar Sep 24 '20 19:09 martinbydefault

I can't say why it might have been removed 8-10 years ago. If someone feels like reviving it I'm good with that. The 3.0 entry is awfully sparse on useful information.

kingthorin avatar Sep 24 '20 19:09 kingthorin

I will try and review the content (from v3) and cross reference it with other locations in the guide to see where it was ported. Pretty doubtful it has been removed, more so renamed or rewritten. Should have an answer by early next week (unless I take a surprise vacation)

ThunderSon avatar Sep 25 '20 08:09 ThunderSon

Weird. @MatOwasp might you know more on this. I was not able to find anything on this matter.

ThunderSon avatar Sep 25 '20 09:09 ThunderSon

Yes, I remember we decided to drop it because not really executed during a test.

MatOwasp avatar Sep 25 '20 09:09 MatOwasp

Reference testbed:

  • http://racetheweb.io/bank
  • https://github.com/insp3ctre/race-the-web

kingthorin avatar Sep 25 '20 13:09 kingthorin

@martinbydefault any interest in tackling this?

kingthorin avatar Sep 25 '20 13:09 kingthorin

@kingthorin Sorry for the delay in the answer. At the moment I'm not able to tackle this but will try to do it in the near future. I also suggested it in the pool of collaboration ideas of my local OWASP chapter to see if anyone wants to take it sooner than me.

martinbydefault avatar Nov 05 '20 04:11 martinbydefault

Please comment if you are still working on this issue, as it has been inactive for 30 days. To give everyone a chance to contribute, we are releasing it to new contributors.

github-actions[bot] avatar Dec 15 '20 00:12 github-actions[bot]

Please comment if you are still working on this issue, as it has been inactive for 30 days. To give everyone a chance to contribute, we are releasing it to new contributors.

github-actions[bot] avatar Apr 15 '21 02:04 github-actions[bot]

Please comment if you are still working on this issue, as it has been inactive for 30 days. To give everyone a chance to contribute, we are releasing it to new contributors.

github-actions[bot] avatar Jun 15 '21 00:06 github-actions[bot]

Please comment if you are still working on this issue, as it has been inactive for 90 days. To give everyone a chance to contribute, we are releasing it to new contributors.

github-actions[bot] avatar Oct 15 '21 00:10 github-actions[bot]

Hi everyone, Should we add this test case (Test case for Race Condition) into section "4.10 Business Logic Testing"?


Edit: Yes adding it under business logic makes sense. Whatever the new ref should be.

lightangel1412 avatar Nov 30 '21 05:11 lightangel1412

That seems reasonable to me.

kingthorin avatar Nov 30 '21 18:11 kingthorin

https://hackerone.com/reports/1540969

kingthorin avatar Jan 09 '23 11:01 kingthorin