Changeset 48429 for trunk/src/wp-includes/feed.php

Timestamp:

07/10/2020 10:21:22 PM (4 years ago)

Author:

whyisjake

Message:

Feeds: Ensure that enclosures produce valid XML.

Metadata that is stored on newlines has the possibility of missing values, so rather then coercing values, we can check for them and then implicity set the values.

Fixes #33591.
Props jonnybot, stevenkword, vtieu, birgire, SergeyBiryukov, davidbaumwald, rebasaurus, whyisjake.

File:

• trunk/src/wp-includes/feed.php (modified) (2 diffs)

trunk/src/wp-includes/feed.php

@@ -520 +520 @@
             foreach ( (array) $val as $enc ) {
                 $enclosure = explode( "\n", $enc );
+
+                $url    = '';
+                $type   = '';
+                $length = 0;
+
+                $mimes = get_allowed_mime_types();
+
+                // Parse url
+                if ( isset( $enclosure[0] ) && is_string( $enclosure[0] ) ) {
+                    $url = trim( $enclosure[0] );
+                }
+
+                // Parse length and type
+                foreach ( range( 1, 2 ) as $i ) {
+                    if ( isset( $enclosure[ $i ] ) ) {
+                        if ( is_numeric( $enclosure[ $i ] ) ) {
+                            $length = trim( $enclosure[ $i ] );
+                        } elseif ( in_array( $enclosure[ $i ], $mimes ) ) {
+                            $type = trim( $enclosure[ $i ] );
+                        }
+                    }
+                }
+
+                $html_link_tag = sprintf(
+                    "<link href=\"%s\" rel=\"enclosure\" length=\"%d\" type=\"%s\" />\n",
+                    esc_url( $url ),
+                    esc_attr( $length ),
+                    esc_attr( $type )
+                );
                 /**
                  * Filters the atom enclosure HTML link tag for the current post.
@@ -527 +556 @@
                  * @param string $html_link_tag The HTML link tag with a URI and other attributes.
                  */
-                echo apply_filters( 'atom_enclosure', '<link href="' . esc_url( trim( $enclosure[0] ) ) . '" rel="enclosure" length="' . absint( trim( $enclosure[1] ) ) . '" type="' . esc_attr( trim( $enclosure[2] ) ) . '" />' . "\n" );
+                echo apply_filters( 'atom_enclosure', $html_link_tag );
             }
         }