Skip to content

Tld generator removed - dep update fixes#455

Open
Collingr wants to merge 2 commits intounic:developfrom
Collingr:tld-generator-removed
Open

Tld generator removed - dep update fixes#455
Collingr wants to merge 2 commits intounic:developfrom
Collingr:tld-generator-removed

Conversation

@Collingr
Copy link

Im going to attempt to get this to java21 for LTS.
to get here I'm dropping tld-generator as its dead weight, project is abandoned by google and will not get 21 support.

Fixes the haphazard dependabot updates that broke tests.

Google tld is abandonware. it is not java 11+ compatible.
dependabot moved org.apache.felix:org.apache.felix.webconsole from 4.2.2 to 4.9.10

There are functional changes to the api the were not covered.
AbstractWebConsolePlugin now expects a RequestVariableResolver on the request before rendering HTML.

Old behavior (4.2.2):
startResponse() could run without a resolver. It either used a default or skipped variable resolution when the attribute was missing.

New behavior (4.9.10):
startResponse() calls r.put(...) on the resolver. If the attribute is missing, r is null and you get a NullPointerException.

This is only used in the tests, so the test has been updated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant