fix(deps): update dependency org.jasig.portlet.notification:notification-portlet-api to v4.8.4#359
Open
renovate[bot] wants to merge 1 commit into
Conversation
…ion-portlet-api to v4.8.4
63a342b to
02867e9
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR contains the following updates:
4.8.2→4.8.4Release Notes
Jasig/NotificationPortlet (org.jasig.portlet.notification:notification-portlet-api)
v4.8.4: NotificationPortlet v4.8.4Compare Source
Two-fix patch on top of v4.8.3.
Fixes
Pin
jjwt-implandjjwt-jacksonat runtime (#695)Every request to
/NotificationPortlet/api/v2/notificationscame back 403 Forbidden in 4.8.x deployments — even with a valid uPortal-issued OIDC bearer token. The<notification-icon>and<notification-list>web components both render off this endpoint, so they silently surfaced as empty in the portal.At INFO log level,
SoffitApiPreAuthenticatedProcessingFilteronly said "The following Bearer token is unusable". Turning the package up to DEBUG revealed the real cause:The deployed WAR shipped only
jjwt-api-0.11.5.jar. JJWT 0.10+ split into three jars —jjwt-api(interfaces),jjwt-impl(runtime implementation, whereDefaultJwtParserlives),jjwt-jackson(JSON parsing).uPortal-soffit-core(the chain that brings JJWT into this project) declares the impl and jackson jars asruntimeOnly. Gradle does not propagateruntimeOnlytransitive dependencies through the deprecatedcompileconfiguration this project still uses, so neither impl nor jackson reached this WAR's runtime classpath.Jwts.parser()threw on first call, the filter swallowed it as "Bearer token is unusable", Spring Security returned 403.Fixed by declaring
runtimeOnly \"io.jsonwebtoken:jjwt-impl:\${jjwtVersion}\"andruntimeOnly \"io.jsonwebtoken:jjwt-jackson:\${jjwtVersion}\"directly onnotification-portlet-webapp, withjjwtVersion=0.11.5pinned ingradle.propertiesto match the versionuPortal-soffit-coreships transitively.Bump
uPortalVersionto 5.17.5 (#694)Aligns with the 2026-05 fleet wave and picks up the Bootstrap 5 include de-duplication in the respondr skin (uPortal v5.17.5).
Upgrade notes
No configuration changes required. Drop-in replacement for v4.8.3.
v4.8.3: NotificationPortlet 4.8.3Compare Source
Patch release — single fix on top of v4.8.2.
Fix
The obsolete
ch.qos.logback.classic.selector.servlet.ContextDetachingSCLlistener has been removed fromnotification-portlet-webapp/src/main/webapp/WEB-INF/web.xml(PR #693). The class was part of Logback's J2EE selector machinery and was removed in Logback 1.3.x.Today the listener still loads cleanly because NotificationPortlet bundles Logback 1.1.11 (pinned by Spring Boot 1.5.22, which is itself EOL since 2019). But every other portlet in the fleet that bumped Logback to 1.3+ during the 2026-05 fleet release wave hit
ClassNotFoundExceptionat context startup with this exact listener — and NotificationPortlet would hit the same wall the moment its Spring Boot or Logback is bumped.The listener also has no functional value here. The original loggingSeparation guidance (the URL in the deleted comment) was for using
LoggerContextSelectorwith hot-deploy of separate webapps, which is not how this portal's logging is set up.Compatibility
Out of scope
Configuration
📅 Schedule: (UTC)
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.