Skip to content

Commit

Permalink
fix: set ResolveObject reason (#375)
Browse files Browse the repository at this point in the history
<!-- Please use this template for your pull request. -->
<!-- Please use the sections that you need and delete other sections -->

## This PR
<!-- add the description of the PR here -->

- Reinstate setting of ResolveObject's reason

Discovered as [integration tests
PR](#312) started failing. In
future this kind of thing will be picked up as a pipeline failure (all
the more reason to get to the bottom of the snyk security issue).

### Related Issues
<!-- add here the GitHub issue that this PR resolves if applicable -->

### Notes
<!-- any additional notes for this PR -->

### Follow-up Tasks
<!-- anything that is related to this PR but not done here should be
noted under this section -->
<!-- if there is a need for a new issue, please link it here -->

### How to test
<!-- if applicable, add testing instructions under this section -->

Signed-off-by: Skye Gill <gill.skye95@gmail.com>
  • Loading branch information
skyerus authored Feb 6, 2023
1 parent e77950b commit dcf199d
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions pkg/service/connect_service.go
Original file line number Diff line number Diff line change
Expand Up @@ -405,6 +405,7 @@ func (s *ConnectService) ResolveObject(
variant,
reason,
))
res.Msg.Reason = reason
res.Msg.Value = val
res.Msg.Variant = variant
return res, nil
Expand Down

0 comments on commit dcf199d

Please sign in to comment.