Drop simple-json dependency in favour of gson

It looks like gson is going to be required for twitter4j and mastodon4j anyway, so let's drop the extra dependency by rewriting the Onionoo client to use gson instead.

added actualpoints::1 component::archived/metrics bot owner::irl points::1 priority::low resolution::fixed severity::normal status::closed type::enhancement labels

FWIW, we're using Gson in the other metrics code bases, too. That doesn't mean that it's the best library around, but at least we didn't run into major problems with it yet.

Regarding your statement to rewrite the Onionoo client, I was wondering if we should make Onionoo's document classes available in metrics-lib, so that it's not necessary anymore to re-implement them. See #24036 (moved) for my thoughts on that, in particular footnote (*8) there. Not sure if this is a great idea, and it might not happen soon enough for your needs. But maybe worth thinking about for the future.

Will be able to add more tomorrow, but quickly. metrics-bot only uses the details document type currently. I plan to add more functionality to the IRC component that would use other document types but this is not a priority at the moment, so if details documents were in metrics-lib then that would be great.

The current details documents look like:

https://people.torproject.org/~irl/metrics-bot/org/torproject/metricsbot/tor/package-summary.html

It's also possible that we just later move the org.torproject.metricsbot.tor package to metrics-lib, although this would require separating out the badge and text generating methods e.g. generateBadge()

Currently metrics-bot does not use metrics-base. Maybe it should?

I can update #24036 (moved) with the package names used in metrics-bot if that is useful, but I'd like to finish some more JavaDoc to make sure myself that I'm happy with the organisation of them first. (I did a bunch of refactoring last night implementing the IRC component so I've not had a good pass over how it looks now yet).

(I think most or all of the open questions are already discussed on #24036 (moved). If not, can you please highlight them again here or there? Thanks!)

I am not immediately working on this ticket. Reassigning to metrics-team to indicate this.

Trac:
Owner: irl to metrics-team
Status: new to assigned

Trac:
Points: N/A to 1

I've started work on this ticket, and have pushed a WIP commit to my https://gitweb.torproject.org/user/irl/metrics-bot.git/log/?h=task/24021 branch. As I'm not aware of anything that depends on the metrics-bot API, there are a few changes that I'm probably going to make that would break backward compatibility.

One example would be dealing with exit policy summaries, which currently are represented as a Map<String,String> where they should have at the very least been a Map<String,String[]>.

Trac:
Owner: metrics-team to irl
Status: assigned to accepted

I've rewritten the Onionoo client to now be using Gson.

This was fixed in bb4556e now merged to master and deployed.

Trac:
Status: accepted to closed
Actualpoints: N/A to 1
Resolution: N/A to fixed

closed

changed time estimate to 8h

added 8h of time spent

mentioned in issue #25848 (moved)