GitHub OAuth: use bigger pages to make fewer requests

[humitos]

Currently, we are using the default pagination that GitHub offers, which is 30.
This commit increases the page size to 100 when fetching organization for a user
and repositories for each organizations.

Note we are already using 100 page size when fetching repositories for a user.

Related to #8979

[stsewd]

We should test if

readthedocs.org/readthedocs/oauth/services/base.py

Line 152 in 902590d

resp = self.get_session().get(url, data=kwargs)

works as expected, because requests says we should be using params https://docs.python-requests.org/en/latest/user/quickstart/#passing-parameters-in-urls 🤔

A quick test shows that data is passed as the body, which get requests shouldn't have... we are only using the kwargs for gitlab, maybe it's just a happy coincidence that they handled the params inside the body as query args.

>>> r = requests.get('https://example.com', params={'foo': 'bar'})
>>> r.url
'https://example.com/?foo=bar'
>>> r.request.body
>>> r = requests.get('https://example.com', data={'foo': 'bar'})
>>> r.url
'https://example.com/'
>>> r.request.body
'foo=bar'

Maybe it's just safer to change data to params.

[humitos]

Good catch! This is definitely not working:

In [9]: s = GitHubService.for_user(User.objects.get(username='github'))[0]

In [10]: response = s.get_session().get('https://api.github.com/user/repos', data={'per_page': 1})

In [11]: len(response.json())
Out[11]: 30

In [12]: response = s.get_session().get('https://api.github.com/user/repos', params={'per_page': 1})

In [13]: len(response.json())
Out[13]: 1

We need to use params= to pass them as GET attributes.

[humitos]

@stsewd I did a final QA locally with all the oauth services supported and all of them worked fine with the params=. I think we are ready to merge this PR.