Make WordPress Core

Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#32864 closed defect (bug) (fixed)

Press This: Set explicit user agent

Reported by: kraftbj's profile kraftbj Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 4.3 Priority: normal
Severity: normal Version: 4.2
Component: Press This Keywords:
Focuses: Cc:

Description

WordPress sends a generic "WordPress/$ver" string as the user agent for HTTP requests.

When the bookmarklet is used, Press This does not display the URL entry form, so if a parent site is blocking the generic UA string (pingback spam, etc), then PT will load with no content and not a way to attempt another site.

To help users and other entities monitor, filter, etc traffic from PT, a more specific UA needs to be set.

Marking as a bug as it causes PT to be unusable on certain sites (e.g. Medium.com)

Attachments (1)

32864.patch (1.0 KB) - added by kraftbj 8 years ago.

Download all attachments as: .zip

Change History (4)

@kraftbj
8 years ago

#1 @SergeyBiryukov
8 years ago

  • Milestone changed from Awaiting Review to 4.3

#2 @SergeyBiryukov
8 years ago

  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from new to closed

In 33061:

Press This: Set explicit user agent.

props kraftbj.
fixes #32864.

#3 @dd32
8 years ago

In 33062:

Press This: Rewrite WP_Press_This::fetch_source_html() to utilise the HTTP API better without the need for a local temporary file.
This reverts [33061] but keeps it's custom UA string.

See #32864

Note: See TracTickets for help on using tickets.