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

Default SecurityConfiguration causes warnings in logs #49

Closed
dmitrygusev opened this Issue Oct 20, 2017 · 2 comments

Comments

Projects
None yet
3 participants
@dmitrygusev
Copy link

dmitrygusev commented Oct 20, 2017

@Contribute(HttpServletRequestFilter.class)
@Security
public static void defaultSecurity(OrderedConfiguration<SecurityFilterChain> configuration, SecurityFilterChainFactory factory) {
	configuration.add("ModulesCompressed", factory.createChain("/modules.gz/**").add(factory.anon()).build(), "before:*");
	configuration.add("Modules", factory.createChain("/modules/**").add(factory.anon()).build(), "before:*", "after:ModulesCompressed");
	configuration.add("Assets", factory.createChain("/assets/**").add(factory.anon()).build(), "before:*", "after:Modules");
}
WARN  o.t.s.s.S.SecurityConfiguration - Unable to add 'Modules' as a dependency of 'ModulesCompressed', as that forms a dependency cycle ('ModulesCompressed' depends on itself via 'Modules'). The dependency has been ignored.
WARN  o.t.s.s.S.SecurityConfiguration - Unable to add 'Assets' as a dependency of 'ModulesCompressed', as that forms a dependency cycle ('ModulesCompressed' depends on itself via 'Assets'). The dependency has been ignored.
WARN  o.t.s.s.S.SecurityConfiguration - Unable to add 'Assets' as a dependency of 'Modules', as that forms a dependency cycle ('Modules' depends on itself via 'Assets'). The dependency has been ignored.
@hotzen

This comment has been minimized.

Copy link

hotzen commented May 14, 2018

same here, is there any progress/how-to-fix on that topic?

@kaosko

This comment has been minimized.

Copy link
Member

kaosko commented Jul 2, 2018

Suppose I was trying to be a bit too cute with the ordering rules. There's no specific reason to try to contribute them before everything else, I'll just remove the ordering constraint.

@kaosko kaosko closed this in f9be5e3 Jul 2, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session.