Response code condition setting isn't respected

Here's a patch that force casts the status code into string

changing status to "Needs Review"

If we are going to cast, isn't it better to remove the strict comparison instead?

Although I'm unsure of the reason behind adding the strict comparison initially, personally, I believe that doing strict comparison is a beneficial practice in general.

But if you are casting you are removing any potential benefit of the strict comparison

I believe that doing strict comparison is a beneficial practice in general.

I agree, unless it's Drupal and dealing with integers.

MR fixes 200 response code checks as well.

Merged! We're working on a few other follow ups before making a new release.

Automatically closed - issue fixed for 2 weeks with no activity.