-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Inject dependencies into interceptors #88
Comments
From brian.mccallister on April 10, 2007 23:58:05 Yea! |
From kevinb9n on April 20, 2007 07:52:39 I assume you are referring specifically to struts2? Does this also including making Labels: -Type-Defect Type-Enhancement |
From kevinb9n on April 20, 2007 07:59:49 Ohhh. You mean method interceptors. Never mind. |
From limpbizkit on June 01, 2008 12:40:36 See also: http://tembrel.blogspot.com/2007/09/injecting-method-interceptors-in-guice.html http://groups.google.com/group/google-guice/browse_thread/thread/a2744812a7a2485f Owner: limpbizkit |
From limpbizkit on June 01, 2008 12:48:32 This can be done once issue 202 has been implemented. That feature request adds this API: We might also want to bind interceptors using Keys rather than instances: That way the interceptor is fully in-the-club. Since equivalent functionality can already be accomplished via Labels: -Priority-Medium Priority-Low |
From andres.a.testi on December 17, 2008 06:01:18 Interceptor binding by key would be useful for allowing method interception in Google |
From nzjess on November 20, 2009 20:43:30 Also the use of Binder.requestInjection(Object) precludes constructor injection of interceptor instances :( |
From sberlin on February 19, 2011 12:30:48 Constructor injection can be mimicked using code at https://code.google.com/p/google-guice/wiki/AOP#Injecting_Interceptors . (Essentially, use Binder.getProvider(..) and pass to the interceptor's constructor.) Status: WontFix |
From crazyboblee on April 10, 2007 00:55:39
Just like every other instance.
Original issue: http://code.google.com/p/google-guice/issues/detail?id=88
The text was updated successfully, but these errors were encountered: