nzt4567 / gh_issue_agent

Created for the MI-PYT course @ FIT CTU
Creative Commons Zero v1.0 Universal
0 stars 0 forks source link

I cannot run this #2

Closed hroncok closed 7 years ago

hroncok commented 7 years ago
$ python __main__.py --repo pyvecpusher/test --auth-file ../../gh.cfg --interval 10
Traceback (most recent call last):
  File "__main__.py", line 187, in <module>
    parse_args()
  File "/home/churchyard/tmp/mipyt/env/lib64/python3.5/site-packages/click/core.py", line 716, in __call__
    return self.main(*args, **kwargs)
  File "/home/churchyard/tmp/mipyt/env/lib64/python3.5/site-packages/click/core.py", line 696, in main
    rv = self.invoke(ctx)
  File "/home/churchyard/tmp/mipyt/env/lib64/python3.5/site-packages/click/core.py", line 889, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/churchyard/tmp/mipyt/env/lib64/python3.5/site-packages/click/core.py", line 534, in invoke
    return callback(*args, **kwargs)
  File "__main__.py", line 183, in parse_args
    'output': output})
  File "__main__.py", line 129, in main
    headers['If-None-Match'] = response.headers['etag']  # if 'etag' in response.headers else None
  File "/home/churchyard/tmp/mipyt/env/lib64/python3.5/site-packages/requests/structures.py", line 54, in __getitem__
    return self._store[key.lower()][1]
KeyError: 'etag'
nzt4567 commented 7 years ago

Unable to replicate this but GH API DOC states that "Most responses return an ETag header." -> My bad, should have left the rest of the line uncommented. Fixing now.

kvasntom@macpro 01 % python3 main.py Patched issue 32 - alksjdflasjdflkajslkdf with labels: ['ASAP'] Patched issue 31 - akslkdjflajsdflkajsdf with labels: ['take-a-look-personally'] Patched issue 30 - klajsdfklajdsklfjakldsf with labels: ['take-a-look-personally'] Patched issue 29 - owejrweoiruwoireuo with labels: ['take-a-look-personally'] Patched issue 28 - kfljasldkjflkasjdflkajdf with labels: ['take-a-look-personally'] Patched issue 27 - sjkfhksjdhfkasf with labels: ['take-a-look-personally'] Patched issue 26 - aksdjflkasjdflsdf with labels: ['take-a-look-personally'] Patched issue 25 - werwerwerwerwer with labels: ['take-a-look-personally'] Patched issue 24 - hjfdjhdkjakjfds with labels: ['take-a-look-personally'] Patched issue 23 - vafvasfdsf with labels: ['take-a-look-personally'] Patched issue 22 - fgfgfgfgf with labels: ['possible_bug', 'ASAP'] Patched issue 21 - fasfdsadfasdf with labels: ['take-a-look-personally'] Patched issue 20 - fasdfasdfsdf with labels: ['take-a-look-personally'] Patched issue 19 - fasdfasdfsdf with labels: ['take-a-look-personally'] Patched issue 18 - fasdfasdfsdf with labels: ['take-a-look-personally'] Patched issue 17 - fasdfasdfsdf with labels: ['take-a-look-personally'] Patched issue 16 - dcnsdknjcns with labels: ['take-a-look-personally'] Patched issue 15 - dsfjksklfdklfdslkjfsldkfj with labels: ['take-a-look-personally'] Patched issue 13 - jkdfkjjklljkdsfkjlsdf with labels: ['take-a-look-personally'] Patched issue 12 - jicijdijcdidcijcdijd with labels: ['take-a-look-personally'] Patched issue 11 - klcalksjdclkajsdc with labels: ['take-a-look-personally'] Patched issue 9 - just a tiny bugisek with labels: ['possible_bug'] GitHub output unchanged, no action needed

nzt4567 commented 7 years ago

Should be fixed by https://github.com/nzt4567/mi-pyt/commit/7d4b6f9cdd126106e76c468b13b9621c7ba248ca - let me know if this works for you. Thanks

nzt4567 commented 7 years ago

Ono to nefunguje hlavne proto, ze ty jako repo zadas i uzivatelsky jmeno, kdezto ja jsem vzal edux za slovo uplne exaktne (tj. option na repo je opravdu jenom nazev repa), ale option na uzivatele uz jsem jaksi neudelal (tj. je hardcoded a tudiz tvuj token k nemu nemuze nikdy fungovat) :) Tj. samozrejme nesmysl, uzivatelsky jmeno musi jit zadat - fixnu vecer s druhou verzi, option pro repo bude i pro uzivatelsky jmeno, jako to normalne spoustis v tom nefungujicim vypisu nahore.

Je btw OK, kdyz ta nasazena aplikace na pythonanywhere funguje jenom s mym uzivatelem/tokenem a ty si to se svym uzivatelem/tokenem budes moci ozkouset lokalne?

hroncok commented 7 years ago

To by mělo být ok

nzt4567 commented 7 years ago

Oki, diky. A my apologies, nejak jsem povazoval za dostatecnej k autentizaci ten token a nedoslo mi, ze ten user je toho dost dulezitou soucasti.