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

Webhooks have started to fail with TypeError: __init__() got an unexpected keyword argument 'client_id' #66

Closed
ekohl opened this issue Aug 27, 2024 · 5 comments

Comments

@ekohl
Copy link
Contributor

ekohl commented Aug 27, 2024

At 2024-08-27 14:00:25 I saw the last successful webhook delivery. After that it has started to fail with:

Traceback (most recent call last):
File "/opt/app-root/src/app.py", line 5, in <module>
run_prprocessor_app()
File "/opt/app-root/src/prprocessor/__main__.py", line 513, in run_prprocessor_app
run_app(
File "/opt/app-root/lib64/python3.9/site-packages/octomachinery/app/server/runner.py", line 71, in run
run_until_complete(run_server_forever, config, event_routers)
File "/opt/app-root/lib64/python3.9/site-packages/anyio/__init__.py", line 72, in run
return asynclib.run(func, *args, **backend_options) # type: ignore
File "/opt/app-root/lib64/python3.9/site-packages/anyio/_backends/_asyncio.py", line 114, in run
raise exception
File "/opt/app-root/lib64/python3.9/site-packages/anyio/_backends/_asyncio.py", line 76, in wrapper
retval = await func(*args)
File "/opt/app-root/lib64/python3.9/site-packages/octomachinery/utils/asynctools.py", line 15, in async_func_wrapper
return await async_func(*args, **kwargs)
File "/opt/app-root/lib64/python3.9/site-packages/octomachinery/app/server/machinery.py", line 134, in run_forever
await _prepare_github_app(github_app)
File "/opt/app-root/lib64/python3.9/site-packages/octomachinery/app/server/machinery.py", line 64, in _prepare_github_app
await github_app.log_installs_list()
File "/opt/app-root/lib64/python3.9/site-packages/octomachinery/github/api/app_client.py", line 71, in log_installs_list
installations = await self.get_installations()
File "/opt/app-root/lib64/python3.9/site-packages/octomachinery/github/api/app_client.py", line 133, in get_installations
async for install in amap(
File "/opt/app-root/lib64/python3.9/site-packages/octomachinery/utils/asynctools.py", line 83, in amap
yield await try_await(callback(async_value))
File "/opt/app-root/lib64/python3.9/site-packages/octomachinery/utils/asynctools.py", line 68, in try_await
return await potentially_awaitable
File "/opt/app-root/lib64/python3.9/site-packages/octomachinery/utils/asynctools.py", line 89, in callback_wrapper
return await try_await(callback(**args_dict))
TypeError: __init__() got an unexpected keyword argument 'client_id'

The container was built 25 days ago and runs https://github.com/theforeman/prprocessor. I've seen this happen before when GitHub changed their webhooks, but from the trace it's not obvious to me where it's failing.

Upvote & Fund

  • We're using Polar.sh so you can upvote and help fund this issue.
  • We receive the funding once the issue is completed & confirmed by you.
  • Thank you in advance for helping prioritize & fund our backlog.
Fund with Polar
@MahouShoujoMivutilde

This comment was marked as resolved.

@webknjaz
Copy link
Member

@ekohl thanks, I made a long-term fix 2 days ago: #61.

@ekohl
Copy link
Contributor Author

ekohl commented Aug 28, 2024

Weird that for us it only started to show up yesterday while for you it showed up earlier.

@ekohl
Copy link
Contributor Author

ekohl commented Aug 28, 2024

I just updated to 0.3.11 and it works again. I can see the following warning show up though:

WARNING:octomachinery.utils.asynctools:Excessive arguments passed to callback <class 'octomachinery.github.models.GitHubAppInstallation'>

@webknjaz
Copy link
Member

Weird that for us it only started to show up yesterday while for you it showed up earlier.

Yeah, I think it started showing up in one app a day earlier than in the other. They're probably rolling it out gradually.

I can see the following warning show up though:

That's my long-term fix. A warning can give us time to act and change the model instead of crashing the entire program. I'm yet to update the model, though. I have Sentry integrated, so I will see those warnings reported now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants
@ekohl @webknjaz @MahouShoujoMivutilde and others