[01:16:07] (03PS1) 10Huji: Check a user's rights, not groups, to ascertain permissions [pywikibot/core] - 10https://gerrit.wikimedia.org/r/531589 (https://phabricator.wikimedia.org/T229293) [01:31:04] (03CR) 10Huji: "Here is the sad news: this patch works because we have NOT implemented a group membership check in the must_be decorator :(" [pywikibot/core] - 10https://gerrit.wikimedia.org/r/531589 (https://phabricator.wikimedia.org/T229293) (owner: 10Huji) [01:49:37] (03PS2) 10Huji: Check a user's rights, not groups, to ascertain permissions [pywikibot/core] - 10https://gerrit.wikimedia.org/r/531589 (https://phabricator.wikimedia.org/T229293) [01:50:50] (03CR) 10Huji: "Now we are talking! I enabled the previously defunct check for user rights. Interestingly, another piece of code already attempted to use " [pywikibot/core] - 10https://gerrit.wikimedia.org/r/531589 (https://phabricator.wikimedia.org/T229293) (owner: 10Huji) [03:43:50] (03CR) 10Dalba: Check a user's rights, not groups, to ascertain permissions (032 comments) [pywikibot/core] - 10https://gerrit.wikimedia.org/r/531589 (https://phabricator.wikimedia.org/T229293) (owner: 10Huji) [03:43:57] (03CR) 10Dalba: [C: 04-1] Check a user's rights, not groups, to ascertain permissions [pywikibot/core] - 10https://gerrit.wikimedia.org/r/531589 (https://phabricator.wikimedia.org/T229293) (owner: 10Huji) [04:08:15] (03CR) 10Dalba: [C: 04-1] Check a user's rights, not groups, to ascertain permissions (031 comment) [pywikibot/core] - 10https://gerrit.wikimedia.org/r/531589 (https://phabricator.wikimedia.org/T229293) (owner: 10Huji) [04:14:11] (03CR) 10Dalba: [C: 04-1] Check a user's rights, not groups, to ascertain permissions (031 comment) [pywikibot/core] - 10https://gerrit.wikimedia.org/r/531589 (https://phabricator.wikimedia.org/T229293) (owner: 10Huji) [11:54:41] (03PS50) 10Ankita Mandal: Tooltip texts modified. Bug: T213513 [extensions/Cargo] - 10https://gerrit.wikimedia.org/r/521184 (https://phabricator.wikimedia.org/T213513) [11:59:01] (03PS51) 10Ankita Mandal: Tooltip texts on Cargo Query page modified. Bug: T213513 [extensions/Cargo] - 10https://gerrit.wikimedia.org/r/521184 (https://phabricator.wikimedia.org/T213513) [14:39:42] 10Pywikibot, 10Pywikibot-login.py, 10Patch-For-Review: dualism between user and sysop needs to be overtaken - https://phabricator.wikimedia.org/T71283 (10Huji) I am considering to take this over after T229293 is fixed. Besides site.py is there any other place where we allow dualism? [15:27:48] (03PS52) 10Yaron Koren: Tooltip texts on Cargo Query page modified. Bug: T213513 [extensions/Cargo] - 10https://gerrit.wikimedia.org/r/521184 (https://phabricator.wikimedia.org/T213513) (owner: 10Ankita Mandal) [16:45:05] (03CR) 10Huji: Check a user's rights, not groups, to ascertain permissions (031 comment) [pywikibot/core] - 10https://gerrit.wikimedia.org/r/531589 (https://phabricator.wikimedia.org/T229293) (owner: 10Huji) [16:48:50] (03PS3) 10Huji: Check a user's rights, not groups, to ascertain permissions [pywikibot/core] - 10https://gerrit.wikimedia.org/r/531589 (https://phabricator.wikimedia.org/T229293) [16:50:48] (03CR) 10jerkins-bot: [V: 04-1] Check a user's rights, not groups, to ascertain permissions [pywikibot/core] - 10https://gerrit.wikimedia.org/r/531589 (https://phabricator.wikimedia.org/T229293) (owner: 10Huji) [16:55:15] 10Pywikibot, 10Patch-For-Review: invalid CSRF token error shown with each block - https://phabricator.wikimedia.org/T229293 (10Ciencia_Al_Poder) I think the bot shouldn't check user rights neither. Do what the bot owner has instructed it to do, and if you can't by permissions, the api will return a relevant er... [19:17:55] 10Pywikibot, 10Patch-For-Review: invalid CSRF token error shown with each block - https://phabricator.wikimedia.org/T229293 (10Huji) @Ciencia_Al_Poder I agree with you in essence. All of these checks are a side effect of the fact that we have historically allowed a user to run *one* bot script with a configur... [20:51:44] 10Pywikibot, 10Patch-For-Review: invalid CSRF token error shown with each block - https://phabricator.wikimedia.org/T229293 (10Dvorapa) Yes, the functionality is archaic and for todays needs of sysops (one sysop account, one bot account) it does not work well anyway (Pywikibot should support some //easy// swit... [23:58:28] (03CR) 10Zhuyifei1999: Check a user's rights, not groups, to ascertain permissions (031 comment) [pywikibot/core] - 10https://gerrit.wikimedia.org/r/531589 (https://phabricator.wikimedia.org/T229293) (owner: 10Huji)