-
Notifications
You must be signed in to change notification settings - Fork 569
Closed
Labels
BugGood first issueGood for newcomersGood for newcomersHelp wantedExtra attention is neededExtra attention is needed
Description
Environment
How do you use Sentry?
Sentry SaaS (sentry.io)
Which SDK and version?
sentry_sdk for python version 1.3.1
Steps to Reproduce
>>> import sentry_sdk
>>> sentry_sdk.VERSION
'1.3.1'
>>> sentry_sdk.init(None, 1)
Traceback (most recent call last):
File "<stdin>", line 1, in <module>
File "/home/emmanuel/projects/parsec-cloud/venv/lib/python3.8/site-packages/sentry_sdk/hub.py", line 105, in _init
client = Client(*args, **kwargs) # type: ignore
File "/home/emmanuel/projects/parsec-cloud/venv/lib/python3.8/site-packages/sentry_sdk/client.py", line 84, in __init__
self.options = get_options(*args, **kwargs) # type: Dict[str, Any]
File "/home/emmanuel/projects/parsec-cloud/venv/lib/python3.8/site-packages/sentry_sdk/client.py", line 51, in _get_options
options = dict(*args, **kwargs)
TypeError: 'int' object is not iterable
>>> sentry_sdk.init(None, 1, 2)
Traceback (most recent call last):
File "<stdin>", line 1, in <module>
File "/home/emmanuel/projects/parsec-cloud/venv/lib/python3.8/site-packages/sentry_sdk/hub.py", line 105, in _init
client = Client(*args, **kwargs) # type: ignore
File "/home/emmanuel/projects/parsec-cloud/venv/lib/python3.8/site-packages/sentry_sdk/client.py", line 84, in __init__
self.options = get_options(*args, **kwargs) # type: Dict[str, Any]
File "/home/emmanuel/projects/parsec-cloud/venv/lib/python3.8/site-packages/sentry_sdk/client.py", line 51, in _get_options
options = dict(*args, **kwargs)
TypeError: dict expected at most 1 argument, got 2Expected Result
Client.__init__ should raise an explicit TypeError exception about the fact it only accept one non-kwargs argument.
Actual Result
Currently the client crash due to an improper usage of dict:
sentry-python/sentry_sdk/client.py
Line 51 in 832263b
| options = dict(*args, **kwargs) |
Metadata
Metadata
Assignees
Labels
BugGood first issueGood for newcomersGood for newcomersHelp wantedExtra attention is neededExtra attention is needed