Changing error code for AuthErrorStates.NotRequired to 406

tableau / TabPy

Execute Python code on the fly and display results in Tableau visualizations:

https://tableau.github.io/TabPy/

MIT License

1.56k stars 598 forks source link

Changing error code for AuthErrorStates.NotRequired to 406 #515

Closed jakeichikawasalesforce closed 3 years ago

jakeichikawasalesforce commented 3 years ago

This change makes the error code distinct for AuthErrorStates.NotRequired, rather than simply a 400 error. This will allow the client (e.g. Tableau) to differentiate between errors more reliably. Adjusted tests accordingly (test_endpoint_handler.py, test_endpoints_handler.py, and test_service_info_handler.py).

This change also bumps the version to 2.4.0, and makes note of the new toggle to turn off evaluate API.

jakeichikawasalesforce commented 3 years ago

Closing merge request to fix failed checks.