Search results

I'm noting that in this test, "unbrekable umbrella" is misspelled. That doesn't seem sufficient to cause test breakage, but might be a small step in the right direction, given the latest observations. I'm also noting that Crimbo Training: Passenger Greeter does provide +1 exp in the modtrace...

Indeed, this is the newer version of nginx correlated with the GOAWAYs, as I thought it might be. That debug log is fascinating, though. It almost looks like we're running into a timeout, since it's just over an hour since you initiated the first request. This might actually be AWS ALB once...

21 brings new preview features in the form of string templates and unnamed patterns / variables, and finalizes record patterns and virtual threads (both introduced in Java 19). By upgrading from 17, we also get better Unicode handling, and a couple of other minor library changes that would...

Bumping this thread -- as far as I'm aware, we are now able to build with Java 21. Spotless is now on 6.23.3, the task errors are now resolved, we're on Gradle 8.8. Most warnings are gone (there's still a warning for lib/darrylbu/util/MenuScroller.java since Object.finalize is marked for...

The command I specifically listed should open a log, make a single (small) request, and then turn off the log, so you can run it both at the start of your session and right after encountering the problem.

I was not going to mention the China angle, but since you mention it: I don't think there's any verification needed to state that Alibaba is a Chinese-owned company headquartered in China, and the vast majority of their employees are also based in China. I am not saying by any means that the...

fastjson2 has me worried because it's hundreds of thousands of lines of Java across thousands of files over thousands of commits, and it has a bunch of features we don't need (read: broad attack surface). Based on the "projects using us" list, I also don't see much adoption by other large...

@fronobulax raised some points in the pull request that I feel warrant more discussion / visibility. @ikzann @Ryo_Sangnoir The obvious place where faster JSON might be relevant is in parsing api.php, but it's also not clear to me that JSON handling is a bottleneck at this time. It's also not...

Hm. For the next time this happens again, mind grabbing a debug log at the start of your session, then again right after it occurs? Specifically, debug on; text game.php; debug off should be enough to grab some response headers, which'll tell us which version of nginx is issuing the GOAWAY...

KoL is apparently using both nginx/1.18.0 and nginx/1.8.1, per the returned headers (the AWSALB cookie pins you to a particular backend). Perhaps they've recently started a migration, which would explain why this started happening again. (HTTP/2 support in nginx was added in 1.9.5.)...

That said, r26388 implemented the nominal "reset the HttpClient every X requests" (which was later refined in r27798). I would be a little surprised if we're running into the headers-too-big problem. I'd be more inclined to believe that something isn't actually using the ResettingHttpClient.

If that were the case, then ~every request would result in a GOAWAY. We do not have our own hand-rolled HTTP codec, as we're relying on Java's standard library implementation. As such, I'm very skeptical of the notion that Mafia is doing something that's standards-noncompliant. We have seen...

As xKiv points out, you're probably best off updating Mafia to a more recent build. That said... The passwords are stored in settings/GLOBAL_prefs.txt as saveState.<your username> as a number, after having gone through a couple of encoding steps. Let's suppose you find...

It looks like the URL has moved to https://museum.loathers.net/api/player/2115655?routes/player.$id._index (as @Ryo_Sangnoir pointed out in May) let kolmafia = require("kolmafia") let results = {}; for (let playerId of [1, 354981]) { let url =...

Hm. I wonder if this is related to being locked in a chain of choice adventures at the time we're trying to refresh our list of skills.

I don't suppose you can share any maximization strings or details re: exactly what got equipped when? If you can find a sequence of actions (beyond "run a specific script that adventures and otherwise depends a lot on current player state") that reproduces the bad state consistently from a...

Anyways, let us know if you still see this problem with r28033 (or later). I can't test readily because I don't think I have access to an account with > 2 billion meat on hand.

CharSheetRequest still uses StringUtilities.parseInt in this and a few related contexts, which results in clobbering the value we parse from api.php / charpane.php. edit: https://github.com/kolmafia/kolmafia/pull/2416

I see the same error on r28030 (i.e. before my dependency bump). It seems nominally benign, although I believe it's tied to the greclipse() line in the spotless stanza.

I saw that too. It's probably related to at least one of the dependencies I bumped. When I tried upgrading spotless, that yielded some actual diffs, so I held off for the time being. I'll look into it some more.