[MAGNOLIA-8652] Fix NPE while doing bearer authentication within magnolia Created: 28/Nov/22  Updated: 23/Oct/23  Resolved: 13/Dec/22

Status: Closed
Project: Magnolia
Component/s: None
Affects Version/s: 6.3.0
Fix Version/s: 6.3.0

Type: Bug Priority: Neutral
Reporter: Dai Ha Assignee: Dai Ha
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
dependency
is depended upon by MGNLREST-546 [e2e] Norsu Delivers references and i... Closed
Template:
Acceptance criteria:
Empty
Task DoD:
[X]* Doc/release notes changes? Comment present?
[X]* Downstream builds green?
[X]* Solution information and context easily available?
[X]* Tests
[X]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
Bug DoR:
[X]* Steps to reproduce, expected, and actual results filled
[X]* Affected version filled
Epic Link: DevX Bucket
Team: DeveloperX

 Description   

Steps to reproduce

  1. Setup saas local webapp
  2. Get bearer authentication token
  3. Try to authenticate and rest endpoint

Result: NPE

java.lang.NullPointerException    info.magnolia.context.UserContextImpl.login(UserContextImpl.java:114)    info.magnolia.context.MgnlContext.login(MgnlContext.java:119)    info.magnolia.sso.SsoLoginFilter.lambda$doFilter$1(SsoLoginFilter.java:99)    org.pac4j.core.engine.DefaultSecurityLogic.perform(DefaultSecurityLogic.java:140)    info.magnolia.sso.SsoLoginFilter.doFilter(SsoLoginFilter.java:81)    info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:85)    info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:79)    info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:81)    info.magnolia.cms.filters.UnicodeNormalizationFilter.doFilter(UnicodeNormalizationFilter.java:89)    info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:85)    info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:79)    info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:81)

Expected results

No NPE occurs

Actual results

NPE occurs

Workaround

No

Development notes



 Comments   
Comment by Dai Ha [ 13/Dec/22 ]

Minor fix, bearer auth now can be used within mgnl webapp, see outcome https://jira.magnolia-cms.com/browse/MGNLREST-546

Generated at Mon Feb 12 04:34:35 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.