WordPress.org

Make WordPress Core

Opened 2 years ago

Last modified 6 days ago

#41404 new enhancement

Sunset Windows Live Writer Manifest File

Reported by: jhabdas Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version: 4.9
Component: General Keywords:
Focuses: Cc:
PR Number:

Description

Given the following code produced on the front-end for users:

<link
  rel="wlwmanifest"
  type="application/wlwmanifest+xml"
  href="http://example.com/wp-includes/wlwmanifest.xml"
>

Following the link to the manifest XML file produces a node in the tree looking like:

<adminUrl>
  <![CDATA[ {blog-postapi-url}/../wp-admin/ ]]>
</adminUrl>

Manifest produced by the wlwmanifest_link() function in general-template.php and was added to core in version 2.3.1 with ticket/5023 10 years ago.

Assuming usage of Windows Live Writer is less than 2% is there anything which would prevent this code from being deprecated in an upcoming minor and (ideally) removed from the codebase altogether?

I suspect removing this from Core will have the following benefits:

  • Improved security by greater obscurity of the admin URL
  • Less code to download by the browser
  • Less code to maintain in Core
  • Less cognitive load on users inspecting generated site source

Attachments (1)

41404.1.patch (2.1 KB) - added by ayeshrajans 2 years ago.

Download all attachments as: .zip

Change History (6)

@ayeshrajans
2 years ago

#1 @ayeshrajans
2 years ago

It's core maintainers decision, but I'd like to put my +1 as well. Windows Live Essentials (which included Live Writer) [reached](https://support.microsoft.com/en-us/help/18614/windows-essentials) its EOL early this year, and is not available to download anymore.

Attached a patch removing the wp_head hook, the manifest file itself, and some icons included.

#2 @Frank Klein
10 months ago

Microsoft does no longer support Windows Live Writer, and the software us also not compatible with Gutenberg.

If this feature should be kept around for Classic Editor users, maybe this code should be moved to this plugin?

#3 @ayeshrajans
10 months ago

Classic Editor certainly promises further backwards compatibility, so completely removing this could have drastic effects. Moving to Classic Editor (which already has an EOL set) sounds like the ideal solution. I'd be happy to submit a patch for that if others agree.

#4 @wtranch
6 days ago

I'd like this to be considered soon as a commitment by WP to be more performant. Literally no one uses WLW, and yet, this PHP code and associated front-end "bloat" is on every WP installation. It was kind for @jhabdas to hope that up to 2% might still be using WLW, but here at the end of 2019, I don't think even that level of generosity for Microsoft's aged tool can be offered. If it was 0.01% of all WordPress installations, I'd be surprised it even has that level of usage.

Regardless, WordPress once stated that core was to build out components that were in use by the majority of users (or for whom the majority of users would benefit). Of course, that ideal has long been side-stepped, but its unclear as to why (or if we need to even delve into the why).

I disagree that removing this would have "drastic effects". It's bloated and unnecessary code. It actually *should* be removed. Leaving this dusty code to settle in the cobwebs of a "classic editor" is not the correct approach, in my opinion. The classic editor is for allowing some people to use the non-Gutenberg approach to creating content. Just because some people, for whatever reasons, want to develop content using the traditional WordPress method does *not* mean that those same people are running dormant and defunct versions of Microsoft's Windows Live Writer.

We need to excise this code from core at once (along with a host of other legacy code that performs little to no function for 99% of WordPress installations). This is a win-win no matter how you slice it.

To be clear, not only should the front-end calls, the API, and admin calls to WLWmanifest be removed, but also the file(s) itself and any and all references to it. Microsoft moved on nearly a decade ago (https://en.wikipedia.org/wiki/Windows_Live_Writer) with its last contribution to WLW's code in 2012. Maybe it's time WordPress moved on, too.

#5 @ayeshrajans
6 days ago

I suppose there's a slight confusion - when I mentioned "drastic efforts", this was a pre-Gutenberg comment about removing the integration with bkogging tools in general. I totally agree with removing WLW, and even submitted the initial patch suggestion for it.

Let's get rid of it for good.

Note: See TracTickets for help on using tickets.