Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

how to not log in for every single edit? #120

Closed
rwst opened this issue Jun 28, 2020 · 4 comments
Closed

how to not log in for every single edit? #120

rwst opened this issue Jun 28, 2020 · 4 comments

Kommentare

@rwst
Copy link
Contributor

rwst commented Jun 28, 2020

The way I use wikibase-cli (call wd with single edits) leads to unneccessary logins to Wikidata which is now discussed in https://phabricator.wikimedia.org/T256533. They cite https://www.mediawiki.org/wiki/API:Login#Additional_notes where we can read:
If you are sending a request that should be made by a logged-in user, add assert=user parameter to the request you are sending in order to check whether the user is logged in. If the user is not logged-in, an assertuserfailed error code will be returned.

Can your code do this between calls of wd? If not, can it be implemented? If not, would batch mode editing resolve it?

@maxlath
Copy link
Owner

maxlath commented Jun 28, 2020

yes, batch mode would resolve this.

Individual calls to wd do re-login every time, as cookies are not stored between sessions. We could consider implementing such a store. Alternatively, we could push people to use batch mode by displaying a warning when doing many successive edits without batch mode

@rwst
Copy link
Contributor Author

rwst commented Jun 29, 2020

Individual calls to wd do re-login every time, as cookies are not stored between sessions.

Something like this would be helpful in the documentation.

@maxlath
Copy link
Owner

maxlath commented Jun 29, 2020

@rwst your pull requests to improve the documentation are welcome.

rwst added a commit to rwst/wikibase-cli that referenced this issue Jun 30, 2020
maxlath added a commit that referenced this issue Jun 30, 2020
add warning about excessive logins (#120)
@maxlath
Copy link
Owner

maxlath commented May 1, 2021

I assume we can consider this addressed by #122 and close for now

@maxlath maxlath closed this as completed May 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants