nopjmp/discord-webhooks

Add error checking to various parts

Opened this issue · 2 comments

To prevent human errors and throwing back sometimes obscure cURL error messages, we should check the parameters to make sure they are valid.

  • URL check with filter api
  • Message length checks
  • Embed validation
  • Find more places for validation

@nopjmp feel free to check my fork of your rep https://github.com/JohnMagdy/discord-communicator (the project is temporary paused)

It's a little difficult to parse out change changes since you keep deleting and uploading through GitHub instead of using git locally when you make changes.

Quite a few additions I see as anti-features in the aspect of static strings of what color is what. It will also be hard for you to merge my code base back into your's if I do update.

Thanks for mentioning your repository though!