| Index: trunk/phase3/includes/HttpFunctions.php |
| — | — | @@ -125,6 +125,9 @@ |
| 126 | 126 | * protocols, because we only want protocols that both cURL |
| 127 | 127 | * and php support. |
| 128 | 128 | * |
| | 129 | + * @fixme this is wildly inaccurate and fails to actually check most stuff |
| | 130 | + * @fixme do we actually intend to have FTP support here? Does it work consistently? |
| | 131 | + * |
| 129 | 132 | * @param $uri Mixed: URI to check for validity |
| 130 | 133 | * @returns Boolean |
| 131 | 134 | */ |
| Index: trunk/phase3/includes/Cookie.php |
| — | — | @@ -62,6 +62,9 @@ |
| 63 | 63 | * A better method might be to use a blacklist like |
| 64 | 64 | * http://publicsuffix.org/ |
| 65 | 65 | * |
| | 66 | + * @fixme fails to detect 3-letter top-level domains |
| | 67 | + * @fixme fails to detect 2-letter top-level domains for single-domain use (probably not a big problem in practice, but there are test cases) |
| | 68 | + * |
| 66 | 69 | * @param $domain String: the domain to validate |
| 67 | 70 | * @param $originDomain String: (optional) the domain the cookie originates from |
| 68 | 71 | * @return Boolean |