You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently we use the raw cloudevents client directly. This can pose many problems in the future
Breaking changes
Complex flows
So instead we should wrap the client which mitigates the two problems listed above, and allows us to add custom patterns ourselves, e.g. hooks. Users should be able to provide their own client if they want, but we should always wrap the client for any SDK.
cc, err:= cdevents.NewCDEventsClient(cdevents.WithPresendHook(func(client cdevents.Client) {
log.Println("Some fancy logging")
})
// and allowing users to pass in their own clientcc, err:=cdevents.NewCDEventsClient(cdevents.CDEventsClientOptions.WithCloudEventsClient(cloudeventsClient))
The text was updated successfully, but these errors were encountered:
Currently we use the raw cloudevents client directly. This can pose many problems in the future
So instead we should wrap the client which mitigates the two problems listed above, and allows us to add custom patterns ourselves, e.g. hooks. Users should be able to provide their own client if they want, but we should always wrap the client for any SDK.
The text was updated successfully, but these errors were encountered: