Merge pull request #349 from nemobis/xmlrevisions

Use GET rather than POST for API requests
pull/371/head
nemobis 4 years ago committed by GitHub
commit 353f4d90a6

@ -199,7 +199,7 @@ def getNamespacesAPI(config={}, session=None):
namespaces = config['namespaces'] namespaces = config['namespaces']
namespacenames = {0: ''} # main is 0, no prefix namespacenames = {0: ''} # main is 0, no prefix
if namespaces: if namespaces:
r = session.post( r = session.get(
url=config['api'], url=config['api'],
params={ params={
'action': 'query', 'action': 'query',
@ -488,7 +488,7 @@ def getXMLHeader(config={}, session=None):
try: try:
if config['api']: if config['api']:
print "Trying the local name for the Special namespace instead" print "Trying the local name for the Special namespace instead"
r = session.post( r = session.get(
url=config['api'], url=config['api'],
params={ params={
'action': 'query', 'action': 'query',
@ -1158,7 +1158,7 @@ def getImageNamesAPI(config={}, session=None):
'format': 'json', 'format': 'json',
'ailimit': 500} 'ailimit': 500}
# FIXME Handle HTTP Errors HERE # FIXME Handle HTTP Errors HERE
r = session.post(url=config['api'], params=params, timeout=30) r = session.get(url=config['api'], params=params, timeout=30)
handleStatusCode(r) handleStatusCode(r)
jsonimages = getJSON(r) jsonimages = getJSON(r)
delay(config=config, session=session) delay(config=config, session=session)
@ -1216,7 +1216,7 @@ def getImageNamesAPI(config={}, session=None):
'iiprop': 'user|url', 'iiprop': 'user|url',
'format': 'json'} 'format': 'json'}
# FIXME Handle HTTP Errors HERE # FIXME Handle HTTP Errors HERE
r = session.post(url=config['api'], params=params, timeout=30) r = session.get(url=config['api'], params=params, timeout=30)
handleStatusCode(r) handleStatusCode(r)
jsonimages = getJSON(r) jsonimages = getJSON(r)
delay(config=config, session=session) delay(config=config, session=session)
@ -1736,9 +1736,9 @@ def checkAPI(api=None, session=None):
# handle redirects # handle redirects
for i in range(4): for i in range(4):
print 'Checking API...', api print 'Checking API...', api
r = session.post( r = session.get(
url=api, url=api,
data={ params={
'action': 'query', 'action': 'query',
'meta': 'siteinfo', 'meta': 'siteinfo',
'format': 'json'}, 'format': 'json'},
@ -2074,7 +2074,7 @@ def saveSiteInfo(config={}, session=None):
print 'Downloading site info as siteinfo.json' print 'Downloading site info as siteinfo.json'
# MediaWiki 1.13+ # MediaWiki 1.13+
r = session.post( r = session.get(
url=config['api'], url=config['api'],
params={ params={
'action': 'query', 'action': 'query',
@ -2085,7 +2085,7 @@ def saveSiteInfo(config={}, session=None):
timeout=10) timeout=10)
# MediaWiki 1.11-1.12 # MediaWiki 1.11-1.12
if not 'query' in getJSON(r): if not 'query' in getJSON(r):
r = session.post( r = session.get(
url=config['api'], url=config['api'],
params={ params={
'action': 'query', 'action': 'query',
@ -2095,7 +2095,7 @@ def saveSiteInfo(config={}, session=None):
timeout=10) timeout=10)
# MediaWiki 1.8-1.10 # MediaWiki 1.8-1.10
if not 'query' in getJSON(r): if not 'query' in getJSON(r):
r = session.post( r = session.get(
url=config['api'], url=config['api'],
params={ params={
'action': 'query', 'action': 'query',

Loading…
Cancel
Save