-
Notifications
You must be signed in to change notification settings - Fork 949
[BUG]: rest api swallows all errors (in order to allow for 404's) #2156
Copy link
Copy link
Closed
Labels
Status: Up for grabsIssues that are ready to be worked on by anyoneIssues that are ready to be worked on by anyoneType: BugSomething isn't working as documentedSomething isn't working as documented
Description
Expected Behavior
currently decode of json fails for certain response types, but since all exceptions that come out of the rest client are ignored there's just null stored in state.
Actual Behavior
check explicitly for the response status (in case you want allow 404 as expected behaviour).
Terraform Version
v6.0.0
Affected Resource(s)
- data_source_github_rest_api
Terraform Configuration Files
No response
Steps to Reproduce
No response
Debug Output
No response
Panic Output
No response
Code of Conduct
- I agree to follow this project's Code of Conduct
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
Status: Up for grabsIssues that are ready to be worked on by anyoneIssues that are ready to be worked on by anyoneType: BugSomething isn't working as documentedSomething isn't working as documented
Type
Projects
Status
✅ Done