delta/python-packages/gitlab.git/tests/unit/test_gitlab_http_methods.py, branch jlvillal/missing_comma github.com: gpocentek/python-gitlab.git fix!: raise error if there is a 301/302 redirection 2021-09-08T16:27:41+00:00 John L. Villalovos john@sodarock.com 2021-05-30T23:31:44+00:00 d56a4345c1ae05823b553e386bfa393541117467 Before we raised an error if there was a 301, 302 redirect but only from an http URL to an https URL. But we didn't raise an error for any other redirects. This caused two problems: 1. PUT requests that are redirected get changed to GET requests which don't perform the desired action but raise no error. This is because the GET response succeeds but since it wasn't a PUT it doesn't update. See issue: https://github.com/python-gitlab/python-gitlab/issues/1432 2. POST requests that are redirected also got changed to GET requests. They also caused hard to debug tracebacks for the user. See issue: https://github.com/python-gitlab/python-gitlab/issues/1477 Correct this by always raising a RedirectError exception and improve the exception message to let them know what was redirected. Closes: #1485 Closes: #1432 Closes: #1477 
Before we raised an error if there was a 301, 302 redirect but only
from an http URL to an https URL.  But we didn't raise an error for
any other redirects.

This caused two problems:

  1. PUT requests that are redirected get changed to GET requests
     which don't perform the desired action but raise no error. This
     is because the GET response succeeds but since it wasn't a PUT it
     doesn't update. See issue:
     https://github.com/python-gitlab/python-gitlab/issues/1432
  2. POST requests that are redirected also got changed to GET
     requests. They also caused hard to debug tracebacks for the user.
     See issue:
     https://github.com/python-gitlab/python-gitlab/issues/1477

Correct this by always raising a RedirectError exception and improve
the exception message to let them know what was redirected.

Closes: #1485
Closes: #1432
Closes: #1477
feat: allow global retry_transient_errors setup 2021-08-29T09:31:18+00:00 Karun Japhet karun@japhet.in 2021-08-28T06:11:59+00:00 3b1d3a41da7e7228f3a465d06902db8af564153e `retry_transient_errors` can now be set through the Gitlab instance and global configuration Documentation for API usage has been updated and missing tests have been added. 
`retry_transient_errors` can now be set through the Gitlab instance and global configuration

Documentation for API usage has been updated and missing tests have been added.
chore: move 'gitlab/tests/' dir to 'tests/unit/' 2021-05-27T04:04:31+00:00 John L. Villalovos john@sodarock.com 2021-05-27T04:04:31+00:00 1ac0722bc086b18c070132a0eb53747bbdf2ce0a Move the 'gitlab/tests/' directory to 'tests/unit/' so we have all the tests located under the 'tests/' directory. 
Move the 'gitlab/tests/' directory to 'tests/unit/' so we have all the
tests located under the 'tests/' directory.