[Bug] Not working - Multi-threading broke it

altcha-org / altcha

GDPR compliant, self-hosted CAPTCHA alternative with PoW mechanism and advanced anti-spam filter.

https://altcha.org

MIT License

241 stars 5 forks source link

[Bug] Not working - Multi-threading broke it #20

Closed rallisf1 closed 3 months ago

rallisf1 commented 3 months ago

After latest 0.2.0 update the solution is always null.

I have confirmed maxnumber is the same between server and client and tried the 1e6 default.

I also tried disabling multi-threading by workers="1", to no avail.

Did I miss something?

ovx commented 3 months ago

Hi, it seems to work fine. The only case, where I always get null is when the maxnumber is smaller than the actual random number generated on the server.

rallisf1 commented 3 months ago

Ok got it. It seems it also fails when using exactly the same number, which is not intuitive.

Server: 1e6 / Client: 1e6 -> fails Server: 1e6 -1 / Client: 1e6 -> verified

ovx commented 3 months ago

Fixed in 0.2.1. Thanks for reporting.