Code review comment for lp:~gesha/linaro-license-protection/997211

Revision history for this message
Georgy Redkozubov (gesha) wrote :

> You have some incorrect indentation in LicenseHelper.
> Furthermore, is "redirectlicensephp" cookie supposed to be set in all three
> possible cases of redirection?

It is strictly needed for 200 and 403 cases but it is also needed to identify 403 status in .htaccess and this can't be done with server variables, so I used the same cookie.

« Back to merge proposal