Skip to content

allow use of old package name for ResourceFileServlet - #355

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

allow use of old package name for ResourceFileServlet#355
copybara-service[bot] merged 3 commits into
mainfrom
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

ludoch commented Mar 20, 2025

Copy link
Copy Markdown
Collaborator

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

ludoch commented Mar 20, 2025

Copy link
Copy Markdown
Collaborator

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
@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