Skip to content

allow use of old package name for ResourceFileServlet#355

Merged
copybara-service[bot] merged 3 commits intomainfrom
migrationForResourceFileServlet
Mar 20, 2025
Merged

allow use of old package name for ResourceFileServlet#355
copybara-service[bot] merged 3 commits intomainfrom
migrationForResourceFileServlet

Conversation

@lachlan-roberts
Copy link
Copy Markdown
Collaborator

Allow com.google.apphosting.runtime.jetty9.ResourceFileServlet to be used for apps on Java 21 runtime using EE8, even though package name has changed to com.google.apphosting.runtime.jetty.ee8.ResourceFileServlet.

Remove DEPRECATED_PACKAGE_NAMES values for EE10, as code changes and redeployment are required to upgrade to EE10.

Signed-off-by: Lachlan Roberts <lachlan.p.roberts@gmail.com>
@ludoch
Copy link
Copy Markdown
Collaborator

ludoch commented Mar 20, 2025

Should we also do this mapping for all servlets and filters in
runtime/runtime_impl_jetty12/src/main/java/com/google/apphosting/runtime/jetty/ee8/* ?

@lachlan-roberts
Copy link
Copy Markdown
Collaborator Author

@ludoch yes we should

Looks like the only other ones are NamedDefaultServlet, NamedJspServlet.

I think I could just do the entire com.google.apphosting.runtime.jetty9 package, I don't think there are any classes with that package for the Java21 runtime.

Signed-off-by: Lachlan Roberts <lachlan.p.roberts@gmail.com>
@lachlan-roberts lachlan-roberts requested a review from ludoch March 20, 2025 03:47
@ludoch
Copy link
Copy Markdown
Collaborator

ludoch commented Mar 20, 2025

Still test issues...

Signed-off-by: Lachlan Roberts <lachlan.p.roberts@gmail.com>
copybara-service Bot pushed a commit that referenced this pull request Mar 20, 2025
…FileServlet

PiperOrigin-RevId: 738673566
Change-Id: I5f50385a427b5d5ca4b77f586f33fd13587aeff5
@copybara-service copybara-service Bot merged commit c1d06d9 into main Mar 20, 2025
16 checks passed
@copybara-service copybara-service Bot deleted the migrationForResourceFileServlet branch March 20, 2025 12:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants