[Bug 59929] Add CheckUser to the grants an OAuth consumer can use
https://bugzilla.wikimedia.org/show_bug.cgi?id=59929 Chris Steipp changed: What|Removed |Added Status|PATCH_TO_REVIEW |RESOLVED Resolution|--- |FIXED --- Comment #12 from Chris Steipp --- The grant is on the proposal form now. Thanks Brad! -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 59929] Add CheckUser to the grants an OAuth consumer can use
https://bugzilla.wikimedia.org/show_bug.cgi?id=59929 --- Comment #11 from Gerrit Notification Bot --- Change 109308 merged by jenkins-bot: Add checkuser OAuth group https://gerrit.wikimedia.org/r/109308 -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 59929] Add CheckUser to the grants an OAuth consumer can use
https://bugzilla.wikimedia.org/show_bug.cgi?id=59929 --- Comment #10 from Gerrit Notification Bot --- Change 109309 merged by jenkins-bot: Message for checkuser OAuth group https://gerrit.wikimedia.org/r/109309 -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 59929] Add CheckUser to the grants an OAuth consumer can use
https://bugzilla.wikimedia.org/show_bug.cgi?id=59929 Gerrit Notification Bot changed: What|Removed |Added Status|NEW |PATCH_TO_REVIEW -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 59929] Add CheckUser to the grants an OAuth consumer can use
https://bugzilla.wikimedia.org/show_bug.cgi?id=59929 --- Comment #9 from Gerrit Notification Bot --- Change 109309 had a related patch set uploaded by Anomie: Message for checkuser OAuth group https://gerrit.wikimedia.org/r/109309 -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 59929] Add CheckUser to the grants an OAuth consumer can use
https://bugzilla.wikimedia.org/show_bug.cgi?id=59929 --- Comment #8 from Gerrit Notification Bot --- Change 109308 had a related patch set uploaded by Anomie: Add checkuser OAuth group https://gerrit.wikimedia.org/r/109308 -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 59929] Add CheckUser to the grants an OAuth consumer can use
https://bugzilla.wikimedia.org/show_bug.cgi?id=59929 --- Comment #7 from Brad Jorsch --- (In reply to comment #6) > My expectation would be that if I have CheckUser and OAuth installed, it > would > just work, without requiring identical configuration that every wiki needs to > set up. But then where do we stop? Why not have OAuth and JoesFabulousExtension "just work" too? Also, while looking at bug 60392, I realized another problem here with regard to wiki farms: even if the other extension itself isn't enabled on a particular wiki, the configuration needs to be present on all of them or granting doesn't show up right. But if the extension isn't installed at all, we don't want to be dumping it in the config because it won't make sense. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 59929] Add CheckUser to the grants an OAuth consumer can use
https://bugzilla.wikimedia.org/show_bug.cgi?id=59929 --- Comment #6 from Kunal Mehta (Legoktm) --- (In reply to comment #5) > Because then you have a weird maze of different files to dig through to try > to > find where some setting is being set. When one extension needs some complex > configuration based on the other then this might make sense, but for > something > simple like this I don't see it. I would just stick a comment saying # Note additional settings might be conditionally set in File.php My expectation would be that if I have CheckUser and OAuth installed, it would just work, without requiring identical configuration that every wiki needs to set up. > > That particular suggestion would also depend on the ordering of the loading > of > the extensions, unless you're doing it in $wgExtensionFunctions which can > have > its own issues. I was thinking that rather than functions using $wgMWOAuthGrantPermissions directly, you could have an intermediate function which adds the conditional ones. But I don't know the extension as well as you do, so I'll defer to your judgement :) -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 59929] Add CheckUser to the grants an OAuth consumer can use
https://bugzilla.wikimedia.org/show_bug.cgi?id=59929 --- Comment #5 from Brad Jorsch --- Because then you have a weird maze of different files to dig through to try to find where some setting is being set. When one extension needs some complex configuration based on the other then this might make sense, but for something simple like this I don't see it. That particular suggestion would also depend on the ordering of the loading of the extensions, unless you're doing it in $wgExtensionFunctions which can have its own issues. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 59929] Add CheckUser to the grants an OAuth consumer can use
https://bugzilla.wikimedia.org/show_bug.cgi?id=59929 Kunal Mehta (Legoktm) changed: What|Removed |Added CC||legoktm.wikipe...@gmail.com --- Comment #4 from Kunal Mehta (Legoktm) --- (In reply to comment #1) > This seems like a bug for Wikimedia / Site requests to have it added to > $wgMWOAuthGrantPermissions in the config for WMF sites. Since checkuser isn't > core MediaWiki, it seems awkward to have OAuth try to handle it. I don't see why OAuth can't handle it. It's fine if extensions know about each other, so just do some kind of if class_exists( 'CheckUser' ) { load CU stuff; } -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 59929] Add CheckUser to the grants an OAuth consumer can use
https://bugzilla.wikimedia.org/show_bug.cgi?id=59929 --- Comment #3 from Brad Jorsch --- (In reply to comment #2) > into InitializeSettings.php Should that be CommonSettings.php, specifically somewhere in https://git.wikimedia.org/blob/operations%2Fmediawiki-config.git/3a1f34f81065764fb787e5290e916df44593c5fa/wmf-config%2FCommonSettings.php#L2523 ? -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 59929] Add CheckUser to the grants an OAuth consumer can use
https://bugzilla.wikimedia.org/show_bug.cgi?id=59929 --- Comment #2 from Chris Steipp --- For a minimal implementation, just add $wgMWOAuthGrantPermissions['checkuser']['checkuser'] = true; into InitializeSettings.php, and make sure the message mwoauth-grant-checkuser gets defined in Ex:WikimediaMessages. The authorization screen will show the grant under the "Miscellaneous activity" grants group, which isn't too bad since the number of potential users is so small. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 59929] Add CheckUser to the grants an OAuth consumer can use
https://bugzilla.wikimedia.org/show_bug.cgi?id=59929 --- Comment #1 from Brad Jorsch --- This seems like a bug for Wikimedia / Site requests to have it added to $wgMWOAuthGrantPermissions in the config for WMF sites. Since checkuser isn't core MediaWiki, it seems awkward to have OAuth try to handle it. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 59929] Add CheckUser to the grants an OAuth consumer can use
https://bugzilla.wikimedia.org/show_bug.cgi?id=59929 Dan Garry changed: What|Removed |Added Priority|Unprioritized |Normal CC||jalexan...@wikimedia.org Severity|normal |enhancement -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l