Skip to content

Adds logging functionality for the webhook client.

Mateusz Brawański requested to merge patch/666 into master

Created by: Neuheit

Summary

This PR fixes a null ref thrown when making requests with the webhook client and adds logging capability for the requests made by the webhook client.

Details

For my bucket hash PR, I forgot to take into account the possibility of the base discord client being null when doing bucket handling (which is the case with the webhook client). This was fixed by simply adding a logger property to the rest client, which will either be assigned to the BaseDiscordClient's or the WebhookClient's.

Users can also change their logger preferences (factory, level, format) through optional parameters in the WebhookClient's constructor.

Changes proposed

  • Fixes null refs being thrown in rest client
  • Adds configurability for the webhook client's logger.

Merge request reports