Code review comment for lp:~gesha/linaro-license-protection/non-standart-http-port

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

> Uhm, I forgot ? after the second group:
>
>
> RewriteCond %{HTTP_HOST} ^([^\:]*)(\:.*)?$
> RewriteRule .* - [E=CO_DOMAIN:%1]
>
> This should work for port 80 as well.

No, because in general HTTP_HOST can be IP address, host name or FQDN and it won't work for second case ;)

« Back to merge proposal