[ISSUE-4] Successful request validation quits access phase

[alexpopero]

No description provided.

[alexpopero]

@ddascal Take a look !

[ddragosd]

I believe this was the intent initially. ngx.exit(ngx.OK) quits the current phase handler continuing to run later phases for the current request. This may indeed pose problems for cases when something else needs to be executed in the access_phase, after the validation. Is this the case you're trying to achieve here ? If yes there may be other places to update as well.

At the same time, in case of validation error, the only way this library stops the request is by calling ngx.exit() with another status. In that case, the remaining logic in access_phase won't be executed.

[alexpopero]

@ddragosd That's exactly what I want. In case validation fails here, it's ok to exit the whole request phase.

[alexpopero]

@ddascal ping

[cristianconstantin]

👍

@ddragosd, ngx.exit() will still be called in case of validation errors, so I don't see any harm in applying this change.