Page MenuHomePhabricator

Investigate if the code of Graphoid uses a proper user agent header
Closed, ResolvedPublic

Description

One of the possible reasons for the issues with Graph extension is that the code would not comply to the new requirements of the Query Service: using a user-agent header to identify tools sending queries.

The first step would be to check on Graphoid's code to check if the tool is sending a user-agent header ; if so, we can exclude this option from the possible reasons.

Timebox to max 4h.

Event Timeline

Change 526442 had a related patch set uploaded (by Ladsgroup; owner: Ladsgroup):
[mediawiki/services/graphoid@master] Improve user agent

https://gerrit.wikimedia.org/r/526442

@Lydia_Pintscher @alaa_wmde So the current user agent of graphoid service is graphoid (yurik at wikimedia). Yurik has left Wikimedia for a couple years I think. I made a patch to fix it but it fails because blubber version 3 is not supported anymore (does it mean we can't merge anything in graphoid right now? @akosiaris knows better)

I looked at hadoop for requests to the query service and couldn't find any request having graphoid mentioned in their UA. Maybe I missed something. Can @Smalyshev double check it against this given UA?

The Graph extension builds the URL but I couldn't find any place that it makes requests but I found that it doesn't hit the internal values, it should be configurable and hit the internal URL in production, that way we can bypass all of these issues plus making it faster by avoiding encryption and TLS handshakes.

So the current user agent of graphoid service is graphoid (yurik at wikimedia)

That should work though. In fact, it's better user agent than many others, even though inaccurate :)

couldn't find any request having graphoid mentioned in their UA.

Hmm this is weird. I will check but in general this would suggest something breaks before it gets to even send the query.

Thanks for checking! I think we achieved the scope of this task, more discussion/investigation should go back to T226250.

@Lydia_Pintscher @alaa_wmde So the current user agent of graphoid service is graphoid (yurik at wikimedia). Yurik has left Wikimedia for a couple years I think. I made a patch to fix it but it fails because blubber version 3 is not supported anymore (does it mean we can't merge anything in graphoid right now? @akosiaris knows better)

It's unmaintained as you pointed out (see T211881 for the glory details). Even if we fixed the blubber file (I am guessing that's where the issue is), deploying anything merged would probably not happen. Plus, there is no one to even review the patches and get them merged right now.

Change 526442 merged by Daniel Kinzler:
[mediawiki/services/graphoid@master] Improve user agent

https://gerrit.wikimedia.org/r/526442

Maintenance_bot moved this task from Incoming to In progress on the User-Ladsgroup board.
Maintenance_bot moved this task from In progress to Done on the User-Ladsgroup board.

> It's unmaintained as you pointed out (see T211881 for the gory details). Even if we fixed the blubber file (I am guessing that's where the issue is), deploying anything merged would probably not happen. Plus, there is no one to even review the patches and get them merged right now.

Per https://phabricator.wikimedia.org/T211881#5509001, graphoid is to be undeployed