Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[skeb] An unexpected error occurred: KeyError - 'completed_at' #3112

Closed
biggestsonicfan opened this issue Oct 28, 2022 · 2 comments
Closed

Comments

@biggestsonicfan
Copy link

Probably a site change. Using dev git build.

gallery-dl --verbose https://skeb.jp/@whimgreen
gallery-dl: Version 1.24.0-dev
gallery-dl: Python 3.10.7 - Linux-6.0.3-1-default-x86_64-with-glibc2.36
gallery-dl: requests 2.28.1 - urllib3 1.26.12
gallery-dl: Starting DownloadJob for 'https://skeb.jp/@whimgreen'
skeb: Using SkebUserExtractor for 'https://skeb.jp/@whimgreen'
urllib3.connectionpool: Starting new HTTPS connection (1): skeb.jp:443
urllib3.connectionpool: https://skeb.jp:443 "GET /api/users/whimgreen/works?role=creator&sort=date&offset=0 HTTP/1.1" 200 11654
urllib3.connectionpool: https://skeb.jp:443 "GET /api/users/whimgreen/works/39 HTTP/1.1" 200 3970
skeb: An unexpected error occurred: KeyError - 'completed_at'. Please run gallery-dl again with the --verbose flag, copy its output and report this issue on https://github.com/mikf/gallery-dl/issues .
skeb: 
Traceback (most recent call last):
  File "/home/rob/.local/pipx/venvs/gallery-dl/lib64/python3.10/site-packages/gallery_dl/job.py", line 82, in run
    for msg in extractor:
  File "/home/rob/.local/pipx/venvs/gallery-dl/lib64/python3.10/site-packages/gallery_dl/extractor/skeb.py", line 31, in items
    response, post = self._get_post_data(user_name, post_num)
  File "/home/rob/.local/pipx/venvs/gallery-dl/lib64/python3.10/site-packages/gallery_dl/extractor/skeb.py", line 80, in _get_post_data
    "completed_at"     : resp["completed_at"],
KeyError: 'completed_at'
@mikf
Copy link
Owner

mikf commented Oct 28, 2022

completed_at seems to be the only field that got removed and now there is nothing left that could be used as a date, from the looks of it.

example.json

@mikf
Copy link
Owner

mikf commented Oct 28, 2022

Removing these lines here fixes the bug:

"completed_at" : resp["completed_at"],
"date" : text.parse_datetime(
resp["completed_at"], "%Y-%m-%dT%H:%M:%S.%fZ"),

mikf added a commit that referenced this issue Oct 29, 2022
'completed_at' is no longer included in API responses
mikf added a commit that referenced this issue Oct 29, 2022
'completed_at' is no longer included in API responses
@mikf mikf closed this as completed Oct 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants