Skip to content

Commit

Permalink
Updating exception message from custom overrides (#2157)
Browse files Browse the repository at this point in the history
Co-authored-by: Adam Sachs <[email protected]>
  • Loading branch information
galvana and adamsachs authored Jan 9, 2023
1 parent e7a6e53 commit a8098e7
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 4 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,7 @@ The types of changes are:
* Remove next-auth from privacy center to fix JS console error [#2090](https://github.com/ethyca/fides/pull/2090)
* Nav bug: clicking on Privacy Request breadcrumb takes me to Home instead of /privacy-requests [#497](https://github.com/ethyca/fides/pull/2141)
* Side nav disappears when viewing request details [#2129](https://github.com/ethyca/fides/pull/2155)
* Improve readability for exceptions raised from custom request overrides [#2157](https://github.com/ethyca/fides/pull/2157)

### Removed

Expand Down
6 changes: 2 additions & 4 deletions src/fides/api/ops/service/connectors/saas_connector.py
Original file line number Diff line number Diff line change
Expand Up @@ -467,15 +467,13 @@ def _invoke_read_request_override(
input_data,
secrets,
) # type: ignore
except Exception:
except Exception as exc:
logger.error(
"Encountered error executing override access function '{}'",
override_function_name,
exc_info=True,
)
raise FidesopsException(
f"Error executing override access function '{override_function_name}'"
)
raise FidesopsException(str(exc))

@staticmethod
def _invoke_masking_request_override(
Expand Down

0 comments on commit a8098e7

Please sign in to comment.