Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Marketo - Access Issue causes NullPointer on publish #2337

Closed
3 tasks
chetanyajain opened this issue Jun 19, 2020 · 1 comment
Closed
3 tasks

Marketo - Access Issue causes NullPointer on publish #2337

chetanyajain opened this issue Jun 19, 2020 · 1 comment
Assignees

Comments

@chetanyajain
Copy link

Required Information

  • AEM Version, including Service Packs, Cumulative Fix Packs, etc: 6.5.4
  • ACS AEM Commons Version: 4.7.0
  • Reproducible on Latest? yes

Expected Behavior

I have authored a page using Marketo component and published. When we access the page in publish instance as an anonymous user the form should load fine.

Actual Behavior

_On publish instance when we access the page the form doesn't load. Upon checking the logs we see Null pointer exception.
19.06.2020 01:06:15.751 ERROR [216.221.228.6 [1592528775723] GET /content/project/us/en/test-marketo-form.html HTTP/1.1] org.apache.sling.scripting.sightly.render.ObjectModel Cannot access method configuration on object com.adobe.acs.commons.marketo.impl.MarketoClientConfigurationManagerImpl@6135062b
java.lang.reflect.InvocationTargetException: null
at sun.reflect.GeneratedMethodAccessor5980.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:498)
at org.apache.sling.scripting.sightly.render.ObjectModel.invokeBeanMethod(ObjectModel.java:418) [org.apache.sling.scripting.sightly.runtime:1.1.0.1_4_0]
at org.apache.sling.scripting.sightly.render.ObjectModel.resolveProperty(ObjectModel.java:128) [org.apache.sling.scripting.sightly.runtime:1.1.0.1_4_0]
at org.apache.sling.scripting.sightly.render.AbstractRuntimeObjectModel.getProperty(AbstractRuntimeObjectModel.java:138) [org.apache.sling.scripting.sightly.runtime:1.1.0.1_4_0]
at org.apache.sling.scripting.sightly.impl.engine.runtime.SlingRuntimeObjectModel.getProperty(SlingRuntimeObjectModel.java:32) [org.apache.sling.scripting.sightly:1.1.2.1_4_0]
at org.apache.sling.scripting.sightly.render.AbstractRuntimeObjectModel.resolveProperty(AbstractRuntimeObjectModel.java:74) [org.apache.sling.scripting.sightly.runtime:1.1.0.1_4_0]
at org.apache.sling.scripting.sightly.apps.platform.components.content.embeddable.marketo.marketo_html.render(marketo_html.java:87)
at org.apache.sling.scripting.sightly.render.RenderUnit.render(RenderUnit.java:50) [org.apache.sling.scripting.sightly.runtime:1.1.0.1_4_0]
at org.apache.sling.scripting.sightly.impl.engine.SightlyCompiledScript.eval(SightlyCompiledScript.java:60) [org.apache.sling.scripting.sightly:1.1.2.1_4_0]
at org.apache.sling.scripting.core.impl.DefaultSlingScript.call(DefaultSlingScript.java:386) [org.apache.sling.scripting.core:2.0.56]
at org.apache.sling.scripting.core.impl.DefaultSlingScript.eval(DefaultSlingScript.java:184) [org.apache.sling.scripting.core:2.0.56]
at org.apache.sling.scripting.core.impl.DefaultSlingScript.service(DefaultSlingScript.java:491) [org.apache.sling.scripting.core:2.0.56]
at org.apache.sling.engine.impl.request.RequestData.service(RequestData.java:552)
at org.apache.sling.engine.impl.filter.SlingComponentFilterChain.render(SlingComponentFilterChain.java:44)
at org.apache.sling.engine.impl.filter.AbstractSlingFilterChain.doFilter(AbstractSlingFilterChain.java:82)
...
Caused by: java.lang.NullPointerException: null
at com.adobe.acs.commons.marketo.impl.MarketoClientConfigurationManagerImpl.getConfiguration(MarketoClientConfigurationManagerImpl.java:56)
... 332 common frames omitted

Steps to Reproduce

  • Author a Marketo form
  • Publish the page
  • Access the form in publish

The Null pointer exception comes because OOB the anonymous user doesn't have access to /conf//settings node. Should the MarketoClientConfigurationManagerImpl not use system user and get the required configurations?

Thank you!

@klcodanr klcodanr self-assigned this Jun 22, 2020
@klcodanr klcodanr added the bug label Jun 22, 2020
@stale
Copy link

stale bot commented Aug 22, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Aug 22, 2020
@stale stale bot closed this as completed Aug 29, 2020
davidjgonzalez added a commit that referenced this issue Dec 13, 2021
* Fixes #2337 Adding Service User for reading /conf on publish

Co-authored-by: Dan Klco <klco@adobe.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants