refactor ghauth handler, get rid of callback style codes

sp3c73r2038 commented 2 years ago

remove return_future and _auth_return_future as they are removed in tornado 6.x
refactor GithubOAuth2Mixin with tornado.gen.coroutine
upgrade tornado to 6.1
support PROXY_HOST and PROXY_PORT in ghauth handler
add pycurl to dev-dependencies because proxy is only supported in tornado.curl_httpclient.CurlAsyncHTTPClient

fixes #128

pallxk commented 2 years ago

tornado should be upgraded in setup.py as well, otherwise it won't work for pip install.

$ pip install .
Processing /home/lxk/w/wiredcraft/pipelines
  Preparing metadata (setup.py) ... done
Requirement already satisfied: certifi==2021.10.8 in /home/lxk/.local/share/virtualenvs/pipelines-XaVGfnz-/lib/python3.9/site-packages (from pipelines==0.2.0a1) (2021.10.8)
Requirement already satisfied: docopt==0.6.2 in /home/lxk/.local/share/virtualenvs/pipelines-XaVGfnz-/lib/python3.9/site-packages (from pipelines==0.2.0a1) (0.6.2)
Requirement already satisfied: dotmap==1.3.26 in /home/lxk/.local/share/virtualenvs/pipelines-XaVGfnz-/lib/python3.9/site-packages (from pipelines==0.2.0a1) (1.3.26)
Requirement already satisfied: filelock==3.4.0 in /home/lxk/.local/share/virtualenvs/pipelines-XaVGfnz-/lib/python3.9/site-packages (from pipelines==0.2.0a1) (3.4.0)
Requirement already satisfied: jinja2==3.0.3 in /home/lxk/.local/share/virtualenvs/pipelines-XaVGfnz-/lib/python3.9/site-packages (from pipelines==0.2.0a1) (3.0.3)
Requirement already satisfied: pyyaml==6.0 in /home/lxk/.local/share/virtualenvs/pipelines-XaVGfnz-/lib/python3.9/site-packages (from pipelines==0.2.0a1) (6.0)
Requirement already satisfied: requests==2.26.0 in /home/lxk/.local/share/virtualenvs/pipelines-XaVGfnz-/lib/python3.9/site-packages (from pipelines==0.2.0a1) (2.26.0)
Requirement already satisfied: schema==0.7.5 in /home/lxk/.local/share/virtualenvs/pipelines-XaVGfnz-/lib/python3.9/site-packages (from pipelines==0.2.0a1) (0.7.5)
Requirement already satisfied: sh==1.14.2 in /home/lxk/.local/share/virtualenvs/pipelines-XaVGfnz-/lib/python3.9/site-packages (from pipelines==0.2.0a1) (1.14.2)
Collecting tornado==5.1.1
  Using cached tornado-5.1.1.tar.gz (516 kB)
  Preparing metadata (setup.py) ... done
Requirement already satisfied: urllib3==1.26.9 in /home/lxk/.local/share/virtualenvs/pipelines-XaVGfnz-/lib/python3.9/site-packages (from pipelines==0.2.0a1) (1.26.9)
Requirement already satisfied: charset-normalizer==2.0.12 in /home/lxk/.local/share/virtualenvs/pipelines-XaVGfnz-/lib/python3.9/site-packages (from pipelines==0.2.0a1) (2.0.12)
Requirement already satisfied: idna==3.3 in /home/lxk/.local/share/virtualenvs/pipelines-XaVGfnz-/lib/python3.9/site-packages (from pipelines==0.2.0a1) (3.3)
Requirement already satisfied: contextlib2==21.6.0 in /home/lxk/.local/share/virtualenvs/pipelines-XaVGfnz-/lib/python3.9/site-packages (from pipelines==0.2.0a1) (21.6.0)
Requirement already satisfied: markupsafe==2.1.1 in /home/lxk/.local/share/virtualenvs/pipelines-XaVGfnz-/lib/python3.9/site-packages (from pipelines==0.2.0a1) (2.1.1)
Building wheels for collected packages: pipelines, tornado
  Building wheel for pipelines (setup.py) ... done
  Created wheel for pipelines: filename=pipelines-0.2.0a1-py3-none-any.whl size=740972 sha256=23a2dc94ee906c493cc8ad65f3b638b2634a9ac9ed715ec6341f0fba2a5f38f1
  Stored in directory: /tmp/pip-ephem-wheel-cache-ektuzdm6/wheels/5e/48/a5/919df1e2cff6abb9bf0bcad8bd9d8ffc07f393439a3b002262
  Building wheel for tornado (setup.py) ... done
  Created wheel for tornado: filename=tornado-5.1.1-cp39-cp39-linux_x86_64.whl size=457389 sha256=8be980da508e4e75ddffe71f74e3ddfd11931d1110f58d848e68c92c34c5c3df
  Stored in directory: /home/lxk/.cache/pip/wheels/20/20/11/70017aae7d3ffd16c4bd2d201de37570d178c9e3904286152c
Successfully built pipelines tornado
Installing collected packages: tornado, pipelines
  Attempting uninstall: tornado
    Found existing installation: tornado 6.1
    Uninstalling tornado-6.1:
      Successfully uninstalled tornado-6.1
Successfully installed pipelines-0.2.0a1 tornado-5.1.1

sp3c73r2038 commented 2 years ago

@pallxk forgot to run pipenv-sync. fixed now.

devo-ps / pipelines

refactor ghauth handler, get rid of callback style codes #131