Not a big deal, but I would consider a subroutine here to also check for nulls for these arguments, potentially throwing some sort of exception in that case -- ideally some sort of 400-level client error.