-
-
Notifications
You must be signed in to change notification settings - Fork 165
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
Add the ability to log Flow callout requests and responses #634
Comments
Hi @jongpie ! I was reviewing this issue and setup a sample with the pokeapi just to test what we get as I haven't use Flows for callouts before. Looks like we get access to a resource with three properties:
I don't think any change is required, maybe support Apex-defined collections in case you want to log response body entirely, as we currently support only a string as the "message" |
@JMercie that's incredibly helpful, thanks so much for testing this out. I like the idea of maybe supporting Apex-defined object/collections, or simply some additional String properties on the current Flow actions - but I think let's hold off on this for now. My thought is that at the moment, Nebula Logger's Flow actions already have several properties on them, so it's already a lot of fields for admins to fill out when configuring the invocable action. Before adding even more properties (to support storing Flow callout data), I want to first explore creating a custom property editor (CPE) LWC for the Flow actions to see if that would provide a better user experience for admins configuring the Flow actions. After that, then I'd like to revisit this issue to see if it does make sense to add some way to store the Flow callout data. |
Flows has the ability to make HTTP callouts with External Services, similar to the Apex classes
HttpRequest
,HttpResponse
, andHttp
that can be used to make callouts.Ideally, Nebula Logger should provide a way to log the Flow callout request and response, similar to the Apex instance methods
LogEntryEventBuilder.setHttpRequestDetails()
andLogEntryEventBuilder.setHttpResponseDetails()
(see discussion #632). However, I haven't use this functionality in Flow, so I'm not sure how exactly this works in Flow, nor what the data types are of the Flow callout request & response. It might "just work", but it might require some changes to Nebula Logger's 3 invocable logging classes -FlowLogEntry
,FlowLogRecordEntry
, andFlowCollectionLogEntry
- to support this.The text was updated successfully, but these errors were encountered: