[14:48:12] anomie: https://phabricator.wikimedia.org/T194166 feels dejavu [14:49:35] Reedy: Duplicate of T36257? [14:49:36] T36257: XSS vulnerability scanner false positives - https://phabricator.wikimedia.org/T36257 [14:49:57] Could be... And hence why it may feel dejavu ;) [14:56:50] Or even more of a duplicate of T143792, looks like the output of the same scanner. [15:12:15] anomie: please have a look at the RevisionStore patch again: https://gerrit.wikimedia.org/r/c/406595/ [15:12:26] ...and the follow up with the refactored tests. [15:12:27] https://gerrit.wikimedia.org/r/c/422443 [15:12:50] I'll probably push a new version of the third one one the chain, the multi-slot one, tomorrow: https://gerrit.wikimedia.org/r/c/416465 [22:53:53] Reedy & legoktm: are we good now re CongressLookup? [22:54:00] I don't think so [22:54:03] I'm writing a comment right now [22:54:13] MaxSem: There's a Kaldari patch outstanding about licensing and json file reduction [22:54:31] if ypu're +1 on it, I'm +2 :P [22:54:35] https://gerrit.wikimedia.org/r/#/c/431995/2/README.md [22:55:19] json file change is fine [22:55:29] Same as Kunal, I'm really not sure on the README [22:55:51] As far as I can tell, it looks like the data is non-free, which means we can't use it