application icon indicating copy to clipboard operation
application copied to clipboard

Published 20 hours ago • verified publisher icon nette

added RejectedRequestException

Open dg opened this issue 8 years ago • 0 comments

  • bug fix? yes (discussion https://github.com/nette/application/commit/212ec4301be07d6ead4a09cc935b4d953bd3e602#commitcomment-19783900)
  • new feature? yes
  • BC break? no

In case that framework rejects request, it throws RejectedRequestException instead of BadRequestException or BadSignalException. The BadSignalException should be deprecated in future.

The main reason is to reach finer granularity for BadRequestException exception.

The naming should be discussed. Is better RejectRequestException or RejectedRequestException? What about constants in RejectedRequestException?

ping @JanTvrdik @matej21

dg avatar Jan 26 '17 14:01 dg