Skip to content

Commit b1ff22c

Browse files
committed
Additional sanitization
1 parent e363ca8 commit b1ff22c

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

wp-includes/rest-api/auth/class-wp-rest-token.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -380,8 +380,8 @@ public function authenticate_refresh_token( $user, WP_REST_Request $request ) {
380380
*/
381381
public function require_token() {
382382
$require_token = true;
383-
$request_uri = isset( $_SERVER['REQUEST_URI'] ) ? sanitize_text_field( $_SERVER['REQUEST_URI'] ) : false;
384-
$request_method = isset( $_SERVER['REQUEST_METHOD'] ) ? sanitize_text_field( $_SERVER['REQUEST_METHOD'] ) : false;
383+
$request_uri = isset( $_SERVER['REQUEST_URI'] ) ? sanitize_text_field( wp_unslash( $_SERVER['REQUEST_URI'] ) ) : false;
384+
$request_method = isset( $_SERVER['REQUEST_METHOD'] ) ? sanitize_text_field( wp_unslash( $_SERVER['REQUEST_METHOD'] ) ) : false;
385385

386386
// User is already authenticated.
387387
$user = wp_get_current_user();
@@ -775,11 +775,11 @@ public function validate_token() {
775775
public function get_auth_header() {
776776

777777
// Get HTTP Authorization Header.
778-
$header = isset( $_SERVER['HTTP_AUTHORIZATION'] ) ? sanitize_text_field( $_SERVER['HTTP_AUTHORIZATION'] ) : false;
778+
$header = isset( $_SERVER['HTTP_AUTHORIZATION'] ) ? sanitize_text_field( wp_unslash( $_SERVER['HTTP_AUTHORIZATION'] ) ) : false;
779779

780780
// Check for alternative header.
781781
if ( ! $header && isset( $_SERVER['REDIRECT_HTTP_AUTHORIZATION'] ) ) {
782-
$header = sanitize_text_field( $_SERVER['REDIRECT_HTTP_AUTHORIZATION'] );
782+
$header = sanitize_text_field( wp_unslash( $_SERVER['REDIRECT_HTTP_AUTHORIZATION'] ) );
783783
}
784784

785785
// The HTTP Authorization Header is missing, return an error.

0 commit comments

Comments
 (0)