vaadin4spring icon indicating copy to clipboard operation
vaadin4spring copied to clipboard

So bad exception handling in VaadinManagedSecurityConfiguration

Open bandris opened this issue 8 years ago • 1 comments

Absorbed exceptions may cause headaches. At least a logger pls.

VaadinManagedSecurityConfiguration.java:

            try {
                applicationContext.getBean(AuthenticationManagerConfigurer.class).configure(auth);
            } catch (Exception ex) {
                throw new IllegalStateException(
                    "No AuthenticationManagerConfigurer found. Either define one, or override the GlobalMethodSecurity.configure(AuthenticationManagerBuilder) method.");
            }

bandris avatar Jan 31 '17 12:01 bandris

A better idea would probably be to include the original exception

FearlessHyena avatar May 12 '17 01:05 FearlessHyena