make sure domain is supported as sanity check in execute #1512
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Seeing this error:
Which occurs in the
catch
block inpackages/agents/lighthouse/src/lib/operations/cartographer.ts
.It seems it's likely being thrown here, in
packages/agents/lighthouse/src/lib/operations/relayer.ts
.I guess we're just not checking beforehand that a domain for a given transfer is supported. While all domains should be supported in theory, we should still catch this case especially for debugging our staging environment / broken configurations.
Type of change
Fire off an error,
DomainNotSupported
, if we don't have a given transfer's destination domain configured.