Rudokhvist / ASF-STM

ASF bot list trade matcher
Apache License 2.0
24 stars 2 forks source link

Use ajax api request to get market hash. #25

Closed Rudokhvist closed 2 months ago

Rudokhvist commented 6 months ago

Changes

Needs extensive testing before merging. If anyone wants to help testing - installation link is here

Bertaz commented 4 months ago

Tested and it seems to work better than master. It finds an additional trade for Ultimate General: Civil War. It's also slightly faster, ajax version took 10m 31s 143ms, while master took 10m 37s 913ms. This was with only 1 tradable badge. I imagine the difference will be higher with more badges/bots. Quickly checked the verbose output and it seems to do everything correctly:

Log output _**very long files**, gzipped_ [2 tradable badges, found 6 matches](https://github.com/Rudokhvist/ASF-STM/files/15145422/newSTM_6trades_1714348470900.log.gz) [previous log with no available matches](https://github.com/Rudokhvist/ASF-STM/files/15145447/newSTM_foundNothing_1713981147990.log.gz)

The warning from #26 is not present in this version. I checked the verbose logs for any personal information. It seems to only lists cards, my timezone and bots' steam profile ids, not even listing my profile id. Please tell me if I missed anything.