New rewrite rule for wp-app.php does not work correctly

[Latz]

AtomPub:

the rewrite rule added in #21509 for wp-app.php does not work correctly since it assumes the URL end with "wp-app.php" (.*wp-app\.php$). Actually a URL could look like this:

​http://localhost/wp-app.php/service

In WP3.4 you're even redirected to this URL. Most applications using AtomPub will therefore use the above URL and receive a 404 error and not the anticipated 403.

Rewrite rule should be changed.

[wonderboymusic]

Yeah... sorry about that... patch attached

[wonderboymusic]

Plugin updated as well: ​http://plugins.trac.wordpress.org/changeset/605278/

[markoheijnen]

Tested it and patch works. Not sure if it makes sense to create a unit test for this.

[nacin]

Rather than .*wp-app\.php/?(.+)?$ I think it should be .*wp-app\.php(/.*)?$, essentially requiring the slash if there are additional characters. That still work fine?

[markoheijnen]

Patch on the feedback of nacin. Still works fine

[nacin]

In 22441:

Update the old wp-app.php rule to allow for a trailing slash followed by endpoint data. props wonderboymusic. fixes #22035.