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

Fix issue where saasContext improperly set if sombra is down #43

Merged

dmattia merged 2 commits into main from dmattia/handle_200_saas_context

Jan 8, 2025

Member

dmattia commented Jan 8, 2025

Closes https://transcend.height.app/T-34615

The go client apparently does not return an error for non-200 responses, so we handle them separately here


          Fix issue where saasContext improperly set if sombra is down

0c25981

height bot commented Jan 8, 2025 •

edited

Loading

This pull request has been linked to and will mark 1 task as "Pending Deploy" when merged:

~~T-34615 Transcend terraform provider is setting saas context to 403 forbidden when sombra-ingress is not setup correctly~~ (unlink task)

💡Tip: You can link multiple Height tasks to a pull request.


          bump versions

dmattia requested review from dipack95, michaelfarrell76 and anotherminh

January 8, 2025 02:12

anotherminh approved these changes

View reviewed changes

dmattia merged commit dc82f39 into main

45 of 46 checks passed

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

Labels

None yet