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.

Sign up for GitHub

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

Jump to bottom

Error in reponse code checking due to operator precedence #2

Open

solarjoe opened this issue Apr 25, 2022 · 1 comment

solarjoe commented Apr 25, 2022

This check does not work as expected as is always True:

if response.status_code == 200 or 201:

https://github.com/SoftwareAG/cumulocity-python-agent/blob/e3f697136584b2d609f88cd6c624f858dd4a0d98/deviceRegistration/newDeviceRegistration.py#L62

Because Python reads it as if (response.status_code == 200) or 201

https://docs.python.org/3/reference/expressions.html#operator-precedence

The text was updated successfully, but these errors were encountered:

Author

solarjoe commented Apr 25, 2022

You would write is as if response.status_code in [200, 201]:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment