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

Tested with local LDAP configurations. #625

Closed
wants to merge 2 commits into
base: master
from

Conversation

Projects
None yet
4 participants
@rconline
Contributor

rconline commented Jan 12, 2016

What is this PR for?

@hayssams @Leemoonsoo Just tested #586 with a local LDAP, works well for me. Included a sample config.

What type of PR is it?

Documentation / Sample testing.

How should this be tested?

Change configs to point to a local/available LDAP, uncomment the parameters, replacing the individual values.

@hayssams

This comment has been minimized.

Show comment
Hide comment
@hayssams

hayssams Jan 12, 2016

Contributor

@rconline
You should probably leave "authcBasic" commented and "anon" uncommented.

Contributor

hayssams commented Jan 12, 2016

@rconline
You should probably leave "authcBasic" commented and "anon" uncommented.

@rconline

This comment has been minimized.

Show comment
Hide comment
@rconline

rconline Jan 12, 2016

Contributor

@hayssams thanks for pointing out. Made the change.

Contributor

rconline commented Jan 12, 2016

@hayssams thanks for pointing out. Made the change.

@Leemoonsoo

This comment has been minimized.

Show comment
Hide comment
@Leemoonsoo

Leemoonsoo Jan 13, 2016

Member

@rconline Thanks a lot for sharing this documentation. CI build failing is irrelevant.
LGTM

Member

Leemoonsoo commented Jan 13, 2016

@rconline Thanks a lot for sharing this documentation. CI build failing is irrelevant.
LGTM

@prabhjyotsingh

This comment has been minimized.

Show comment
Hide comment
@prabhjyotsingh

prabhjyotsingh Jan 13, 2016

Contributor

Tested, LGTM

Contributor

prabhjyotsingh commented Jan 13, 2016

Tested, LGTM

@Leemoonsoo

This comment has been minimized.

Show comment
Hide comment
@Leemoonsoo

Leemoonsoo Jan 14, 2016

Member

Merge if there're no more discussions

Member

Leemoonsoo commented Jan 14, 2016

Merge if there're no more discussions

@asfgit asfgit closed this in 4c0fd96 Jan 14, 2016

@Leemoonsoo Leemoonsoo referenced this pull request Jan 15, 2016

Closed

[HOTFIX] add [urls] section in shiro.ini #639

1 of 1 task complete

asfgit pushed a commit that referenced this pull request Jan 15, 2016

[HOTFIX] add [urls] section in shiro.ini
### What is this PR for?
#625 removed `[urls]` from conf/shiro.ini and that cause following exception on start up
```
ERROR [2016-01-14 20:46:38,646] ({main} EnvironmentLoader.java[initEnvironment]:146) - Shiro environment initialization failed
org.apache.shiro.config.ConfigurationException: Unable to instantiate class [anon] for object named '/**'.  Please ensure you've specified the fully qualified class name correctly.
        at org.apache.shiro.config.ReflectionBuilder.createNewInstance(ReflectionBuilder.java:151)
        at org.apache.shiro.config.ReflectionBuilder.buildObjects(ReflectionBuilder.java:119)
        at org.apache.shiro.config.IniSecurityManagerFactory.buildInstances(IniSecurityManagerFactory.java:161)
        at org.apache.shiro.config.IniSecurityManagerFactory.createSecurityManager(IniSecurityManagerFactory.java:124)
        at org.apache.shiro.config.IniSecurityManagerFactory.createSecurityManager(IniSecurityManagerFactory.java:102)
        at org.apache.shiro.config.IniSecurityManagerFactory.createInstance(IniSecurityManagerFactory.java:88)
        at org.apache.shiro.config.IniSecurityManagerFactory.createInstance(IniSecurityManagerFactory.java:46)
        at org.apache.shiro.config.IniFactorySupport.createInstance(IniFactorySupport.java:123)
        at org.apache.shiro.util.AbstractFactory.getInstance(AbstractFactory.java:47)
        at org.apache.shiro.web.env.IniWebEnvironment.createWebSecurityManager(IniWebEnvironment.java:203)
        at org.apache.shiro.web.env.IniWebEnvironment.configure(IniWebEnvironment.java:99)
        at org.apache.shiro.web.env.IniWebEnvironment.init(IniWebEnvironment.java:92)
        at org.apache.shiro.util.LifecycleUtils.init(LifecycleUtils.java:45)
        at org.apache.shiro.util.LifecycleUtils.init(LifecycleUtils.java:40)
        at org.apache.shiro.web.env.EnvironmentLoader.createEnvironment(EnvironmentLoader.java:221)
        at org.apache.shiro.web.env.EnvironmentLoader.initEnvironment(EnvironmentLoader.java:133)
        at org.apache.shiro.web.env.EnvironmentLoaderListener.contextInitialized(EnvironmentLoaderListener.java:58)
        at org.eclipse.jetty.server.handler.ContextHandler.callContextInitialized(ContextHandler.java:782)
        at org.eclipse.jetty.servlet.ServletContextHandler.callContextInitialized(ServletContextHandler.java:424)
        at org.eclipse.jetty.server.handler.ContextHandler.startContext(ContextHandler.java:774)
        at org.eclipse.jetty.servlet.ServletContextHandler.startContext(ServletContextHandler.java:249)
        at org.eclipse.jetty.server.handler.ContextHandler.doStart(ContextHandler.java:717)
        at org.eclipse.jetty.util.component.AbstractLifeCycle.start(AbstractLifeCycle.java:64)
        at org.eclipse.jetty.server.handler.HandlerCollection.doStart(HandlerCollection.java:229)
        at org.eclipse.jetty.server.handler.ContextHandlerCollection.doStart(ContextHandlerCollection.java:172)
        at org.eclipse.jetty.util.component.AbstractLifeCycle.start(AbstractLifeCycle.java:64)
        at org.eclipse.jetty.server.handler.HandlerWrapper.doStart(HandlerWrapper.java:95)
        at org.eclipse.jetty.server.Server.doStart(Server.java:282)
        at org.eclipse.jetty.util.component.AbstractLifeCycle.start(AbstractLifeCycle.java:64)
        at org.apache.zeppelin.server.ZeppelinServer.main(ZeppelinServer.java:114)
Caused by: org.apache.shiro.util.UnknownClassException: Unable to load class named [anon] from the thread context, current, or system/application ClassLoaders.  All heuristics have been exhausted.  Class could not be found.
        at org.apache.shiro.util.ClassUtils.forName(ClassUtils.java:148)
        at org.apache.shiro.util.ClassUtils.newInstance(ClassUtils.java:164)
        at org.apache.shiro.config.ReflectionBuilder.createNewInstance(ReflectionBuilder.java:144)
        ... 29 more
```

### What type of PR is it?
Hot Fix

### Todos
* [x] - fix

### Is there a relevant Jira issue?
no

### How should this be tested?
just build and start zeppelin with default configuration

### Screenshots (if appropriate)

### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no

Author: Lee moon soo <moon@apache.org>

Closes #639 from Leemoonsoo/hotfix_shiro and squashes the following commits:

7a75c48 [Lee moon soo] Hotfix shiro.ini
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment