WordPress.org
Get WordPress

Make WordPress Core

Opened 10 years ago

Closed 10 years ago

#38817 closed defect (bug) (fixed)

REST API: Store the user_agent by default in the comments API

Reported by: dd32 Owned by: rachelbaker
Priority: normal Milestone: 4.7
Component: Comments Version:
Severity: normal Keywords: has-patch has-unit-tests commit
Cc: Focuses: rest-api

Description

The comments API currently doesn't store the user_agent by default, unless the client specifies it.

As the user_agent field is a user-controllable field, I see no issue with allowing it to be set through the REST API parameters, however believe that it should default to $_SERVER['HTTP_USER_AGENT'] when not specified by the client.

Attachments (2)

38817.diff​ (1.6 KB ) - added by dd32 10 years ago.
38817.2.diff​ (2.3 KB ) - added by rachelbaker 10 years ago.
Added ! empty spacing and a unit test

Download all attachments as: .zip

Change History (5)

@dd32
10 years ago

#1 @jnylen0
10 years ago

  • Keywords needs-unit-tests added

Needs tests and missing an operator space: !empty → ! empty. Otherwise makes sense to me.

@rachelbaker
10 years ago

Added ! empty spacing and a unit test

#2 @rachelbaker
10 years ago

  • Keywords has-unit-tests commit added; needs-unit-tests removed

38817.2.diff​ Adds the space in the empty conditional and also includes a unit test.

#3 @rachelbaker
10 years ago

  • Owner set to rachelbaker
  • Resolution → fixed
  • Status new → closed

In 39287:

REST API: On comment create, fallback to the user_agent header value.

If a user-agent is not explicitly provided in the author_user_agent parameter, fallback to the user_agent value in the request header.

Props dd32, jnylen0, rachelbaker.
Fixes #38817.

Note: See TracTickets for help on using tickets.