Skip to content
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.

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

Error-handling post-processing snippet #1587

Open
bbenligiray opened this issue Dec 5, 2022 · 3 comments
Open

Error-handling post-processing snippet #1587

bbenligiray opened this issue Dec 5, 2022 · 3 comments

Comments

@bbenligiray
Copy link
Member

The user wants a callback when the API call errors. We had this functionality, but it was removed at the design phase.

A simple way of emulating the same feature is having an error-handling post-processing step. Normally, when the API response is non-2XX, the Airnode always calls back fail(). We can have an errorHandlingPostProcessing snippet that only gets invoked when the API returns a non-2XX code and post-processes the API response to call back fulfill() with that.

@aTeoke
Copy link

aTeoke commented Dec 9, 2022

@bbenligiray who is the user and what is the urgency please? Should it be ASAP or can be moved after 1.0 milestone? Thanks

@dcroote
Copy link
Contributor

dcroote commented Dec 9, 2022

This was a user on discord. I would say low priority because the user themselves admitted / recognized "my use-case is very niche" and there are creative workarounds that @bbenligiray mentioned, like a proxy or FailedRequest watcher, that could suffice if the feature is badly needed.

@bbenligiray
Copy link
Member Author

I think we should have this or something analogous at some point, but it shouldn't be included in our foreseeable plans. So I created the issue to document it for the future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants