[18:37:08] Krinkle: re https://gerrit.wikimedia.org/r/c/mediawiki/extensions/CheckUser/+/1115504 . WebRequest::isSafeRequest() was meant for that kind of "are writes not expected" check, though markAsSafeRequest() is no longer called since it was deprecated in 0ee08d66160b0fe001. The only thing I don't like about using WebRequest is the mutability of isSafeRequest(), since something has to tell it what to expect. It's odd for CheckUser to ask [18:37:08] the profiler to get the trx profiler to ask what is expected...TransactionProfiler should probably be split from Profiler. Maybe be its own service. [20:22:58] tgr|away: DALs are live. It's a bit opaque to test. So far Chrome hasn't picked it up yet. I'll try again tomorrow. If nothing by then I'll try to find the exact Google API it makes. [20:23:22] API call*