Include Last-Modified header in the response

[mscottford]

I'm authoring a library, dependency-history-maven that can query HTTP Maven repositories to determine the release history for a particular package. That library is using the Last-Modified header value to determine when a file was released into the repository. I'm also working on an integration of this library into a Maven plugin that uses mrm for its integration tests, so I need mrm to include the Last-Modified header for those tests to work.

[slawekjaranowski]

Is it possible to to add some test for it?

[mscottford]

I didn't see any tests for the servlet, and I wasn't sure the best way to go about adding them.

[slawekjaranowski]

Yes I know, there tests are missing ...
So I only ask, it will be possible to add one.

I see eg, that method doGet has many responsibility, like

• provide settings
• provide data for FileEntry and DirectoryEntry

we can split it to separate method and it will be simple to test

But yes such improvement can be don in separate PR.

[mscottford]

Awesome! Thanks for merging this in. Any estimate on when this will be included in a published release?

[slawekjaranowski]

I don't have plan to release new version now, last was two months ago.

But you can try to incentive me, eg by more improvement in code, documentations, ...