Skip to content

Conversation

@clauspruefer
Copy link
Member

@clauspruefer clauspruefer commented May 2, 2025

Done:

  • Multiple parallel GET requests fixed (still result-order not 100% clean)
  • Add unit test for multiple GET requests + 1 truncated request
  • Replace ThreadHandler "global big exception" (404 result) with file exists check
  • To check if result-order is working correctly uncomment some lines (needs to be investigated)

@clauspruefer clauspruefer added the bug Something isn't working label May 2, 2025
@clauspruefer clauspruefer added this to the Release 0.1 milestone May 2, 2025
@clauspruefer clauspruefer self-assigned this May 2, 2025
@clauspruefer
Copy link
Member Author

Currently implementing (a bit more complicated than expected) HTTP/1.1 GET and POST requests + unit tests. Should be finished apx. 2-3 hours.

@clauspruefer
Copy link
Member Author

Tests for POST + truncated following.

@clauspruefer clauspruefer added release Release related and removed bug Something isn't working labels May 6, 2025
@clauspruefer clauspruefer mentioned this pull request May 6, 2025
7 tasks
@clauspruefer
Copy link
Member Author

clauspruefer commented May 7, 2025

Behaviour now with correct implemented ResultOrder.cpp and working GET | POST httpparser.cpp:

  • Single GET Request (Browser, wget) working
  • Multiple GET Requests (Browser) only first Request will be returned
  • Single POST Request (wget) working, result will be delivered, but crashes (where?)

@clauspruefer
Copy link
Member Author

clauspruefer commented May 12, 2025

Multiple GET

On multiple GET Browser request http://testapp1.local/index.html, index.html and favicon.ico will be returned correctly, http://testapp1.local/falcon-robotics.jpgnot (hangs / timeout).

Single GET

On single GET request http://testapp1.local/falcon-robotics.jpg will be returned correctly.

@clauspruefer clauspruefer merged commit 38735b5 into main May 13, 2025
3 checks passed
@clauspruefer clauspruefer deleted the v0.1rc branch September 20, 2025 11:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release Release related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants