Pattypan should send a correct user agent when making request to commons.wikimedia.org

[reedy]

Following on from #138 and the Phabricator tasks...

Can someone please look and verify pattypan is actually using its own user agent in requests?

pattypan/src/pattypan/Settings.java

Line 46 in 2809ce7

public static final String USERAGENT = NAME + "/" + VERSION + " (https://github.com/yarl/pattypan)";

Looking at some of the code, it does seem there is some setup for it such as surrounding the above, but looking at the logs (from the Wikimedia side), it doesn't seem to result in any entries, which is odd.

vicuna seems to be fine.

[Abbe98]

It appears to be used in the session setup:

pattypan/src/pattypan/Session.java

Line 58 in 2809ce7

WIKI.setUserAgent(Settings.USERAGENT);

[reedy]

Yeah, which is why I've created the task.

The useragent doesn't seem to be making it to Wikimedia's Logstash servers which makes me feel like something in the plumbing isn't quite right here.

I noticed yarl/vicuna@0dc286f where it was done in a few more places, I'm not sure specifically why like that

[Abbe98]

We only call setUserAgent once because we only got one instance of the Wiki.java class.

I will need to have a look when I got a debugger at hand, I assume that WIKI.setUserAgent(Settings.USERAGENT); is either dead code or we lose the state of WIKI somewhere.

[Abbe98]

The initialization of the Wiki.java session in Session.java was overwritten by the one in Main.java. I removed the one in Session.java as it would end up being overwritten anyway. Marking this for the 22.03 release.

[reedy]

Yay. Thanks!