snoopdave opened a new pull request, #198: URL: https://github.com/apache/roller/pull/198
## Summary Maps `/roller-services/app/*` to a new `RollerAtomServlet`, a small subclass of Propono's `AtomServlet`, so Roller controls how AtomPub requests are handled before Propono sees them. - **Honour `webservices.enableAtomPub` on every request.** Until now only the service document checked the setting. With AtomPub off, the endpoint now answers 404, the same way the XML-RPC endpoint does. - **Read Atom entry bodies with Roller's standard XML parser configuration.** For a POST of Atom content or a PUT to an entry URI, the servlet reads the body (up to 10 MB), parses it with the same `SafeSAXBuilder` settings Roller uses elsewhere, and hands Propono a buffered copy. An entry that does not parse gets 400; an oversized one gets 413. Media uploads and reads are unchanged. - `RollerAtomHandler`'s entry-URI rule moves into a static `isEntryPath` helper, so the servlet and the handler use the same test. ## Testing - New `RollerAtomServletTest` (10 tests): disabled service, pass-through of reads and media, buffered forwarding of a normal entry, refused entries on POST and PUT, size limit, and the shared entry-path rule. - `mvn -pl app test` on JDK 11: 335 tests, 0 failures, 1 skipped. Targets `roller-6.1.x` for 6.1.7. A matching change for `master` will follow. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
