Make WordPress Core

Changes between Initial Version and Version 1 of Ticket #16940, comment 11


Ignore:
Timestamp:
05/07/2015 03:23:31 AM (10 years ago)
Author:
kraftbj
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #16940, comment 11

    initial v1  
    88}}}
    99
    10 Since the bookmarklet and the direct entry form passes a complete URL, including protocol, as the `u` value, it is including http/https in the URL, which is triggering this rule. Testing a direct URL, changing the u to remove http would avoid the 403 error, but break the feature of PT since `_limit_url` returns `''` if `! preg_match( '/^([!#$&-;=?-\[\]_a-z~]|%[0-9a-fA-F]{2})+$/'` ([[https://core.trac.wordpress.org/browser/tags/4.2.2/src/wp-admin/includes/class-wp-press-this.php#L372|ref]]).
     10Since the bookmarklet and the direct entry form passes a complete URL, including protocol, as the `u` value, it is including http/https in the URL, which is triggering this rule.
     11
     12Testing a direct URL, changing the u to remove http does avoid the 403 error on HostGator, but break the feature of PT since `_limit_url` returns `''` if `! preg_match( '/^([!#$&-;=?-\[\]_a-z~]|%[0-9a-fA-F]{2})+$/'` ([[https://core.trac.wordpress.org/browser/tags/4.2.2/src/wp-admin/includes/class-wp-press-this.php#L372|ref]]).
    1113
    1214I'm thinking of if there is ways around this that suitably work; open to suggestions.