Last modified: 2014-08-26 19:31:02 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T38078, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 36078 - ApiTokens should be a query module
ApiTokens should be a query module
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
API (Other open bugs)
1.20.x
All All
: Low normal (vote)
: ---
Assigned To: Brad Jorsch
:
Depends on: 45199
Blocks:
  Show dependency treegraph
 
Reported: 2012-04-18 21:55 UTC by Bergi
Modified: 2014-08-26 19:31 UTC (History)
15 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
gerrit patch set (114 bytes, text/plain)
2012-04-19 16:51 UTC, Bergi
Details

Description Bergi 2012-04-18 21:55:41 UTC
relating to Bug 27757

The idea was good to add a possibilty for retrieving tokens without specifying a page. Still, it should be a query module instead of an extra action.

An advantage would be that this allows retrieving information about the user and the wiki in only one request, as you can combine queries.

Tokens are clearly a meta property ([[mw:API:Meta]]), I suppose they could even be integrated into the userinfo module (where they'd belong also logically - retrieving information about the currently logged-in user).
Comment 1 Marcin Cieślak 2012-04-19 09:25:46 UTC
I have a question to the whole concept, asking in bug 27757 :)
Comment 2 Bergi 2012-04-19 16:51:00 UTC
Created attachment 10440 [details]
gerrit patch set
Comment 3 Umherirrender 2012-04-21 13:59:23 UTC
You can make easy links with "gerrit<space><changenumber>": Gerrit change #5300
Comment 4 Bergi 2012-04-23 16:28:19 UTC
(In reply to comment #3)
> You can make easy links with "gerrit<space><changenumber>": Gerrit change #5300

OK, but why does [[mw:How to become a MediaWiki hacker#Committing]] state "Post a link to your Gerrit changeset _as_an_attachment_ to the appropriate bug report in Bugzilla"?
Comment 5 Umherirrender 2012-04-28 09:23:15 UTC
(In reply to comment #4)
> (In reply to comment #3)
> > You can make easy links with "gerrit<space><changenumber>": Gerrit change #5300
> OK, but why does [[mw:How to become a MediaWiki hacker#Committing]] state "Post
> a link to your Gerrit changeset _as_an_attachment_ to the appropriate bug
> report in Bugzilla"?

That is a good question. I am not sure, why that is there and why bugs should mark with "patch", when the patch is already commited to gerrit.

I have add a hint to gerrit link syntax for bugzilla[1].

[1] https://www.mediawiki.org/w/index.php?title=How_to_become_a_MediaWiki_hacker&diff=530342&oldid=518421
Comment 6 Bergi 2012-05-07 17:39:05 UTC
Why was this reverted in Gerrit change #6596 ?
Comment 7 Beau 2012-05-07 17:43:37 UTC
It was not reverted. Your change is still awaiting review and merge. The change you linked moves single token (options) to action=tokens.
Comment 8 Marcin Cieślak 2012-06-14 21:20:59 UTC
This issue has been raised in https://bugzilla.wikimedia.org/show_bug.cgi?id=27757#c16:

"I have a question here - currently the token is session-based, but if no longer
bind the token to the target page (edit|delete|protect|move) or the user
(block|unblock), aren't we losing possibility of ever introducing a
notification/conflict resolution/etc. mechanism to notify ("someone is working
on this right now") - kind of "soft advisory locking" on an object?

FlaggedRevs is trying to do this to show that somebody is reviewing the page
right now."

There is also a comment in the code review from Anomie:

"But I think we really need to take a step back and decide just what the model for each kind of token really is, and make it available from the place where it makes the most sense. For example, the token for action=edit currently depends just on the user/session. But do we want to basically "reserve the right" to make it depend on the page being edited? If so, it should stay in prop=info and not be added here; if not, add it here *and* deprecate it in prop=info."

I think this should be addressed/discussed before committing this change.
Comment 9 Yuri Astrakhan 2013-03-04 05:57:08 UTC
Overall token discussion is in bug 45199.
Comment 10 Brad Jorsch 2014-08-26 19:31:02 UTC
Gerrit change #153110

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links