Feature Request: MicroProfile-config integration: Allow injecting annotations with jakarta.inject.Qualifier about guice HOT 12 CLOSED

FYI - nasty reflection was:

package com.google.inject.internal;

import com.google.inject.internal.Annotations.AnnotationChecker;
import org.eclipse.microprofile.config.inject.ConfigProperty;

import java.lang.annotation.Annotation;
import java.util.Collections;

public class ConfigPropertyOrAnnotationChecker extends AnnotationChecker {

    private final AnnotationChecker delegate;

    /**
     * Constructs a new checker that looks for annotations of the given types.
     *
     * @param delegate delegate checker
     */
    public ConfigPropertyOrAnnotationChecker(Object delegate) {
        super(Collections.singleton(ConfigProperty.class));
        this.delegate = (AnnotationChecker) delegate;
    }

    @Override
    boolean hasAnnotations(Class<? extends Annotation> annotated) {
        return annotated == ConfigProperty.class || delegate.hasAnnotations(annotated);
    }
}

and

        try {
            Field bindingAnnotationCheckerField = Annotations.class.getDeclaredField("bindingAnnotationChecker");
            bindingAnnotationCheckerField.setAccessible(true);
            Field modifiersField = Field.class.getDeclaredField("modifiers");
            modifiersField.setAccessible(true);
            modifiersField.setInt(
                bindingAnnotationCheckerField, bindingAnnotationCheckerField.getModifiers() & ~Modifier.FINAL);
            Object bindingAnnotationChecker = bindingAnnotationCheckerField.get(Annotations.class);
            bindingAnnotationCheckerField.set(
                Annotations.class, new ConfigPropertyOrAnnotationChecker(bindingAnnotationChecker));
            bindingAnnotationCheckerField.setAccessible(false);
        } catch (NoSuchFieldException | IllegalAccessException e) {
            throw new ConfigurationException(Collections.singleton(new Message(e.getLocalizedMessage())));
        }

from guice.

Field + Construct interceptors will be a much better option here, Qualifier should already be there - Strictly speaking this should actually be more than do-able

I'll join you on this one

from guice.

TypeListeners seem to be the best route

from guice.

TypeListeners seem to be the best route

Sorry not really sure how they help... the point is to make things annotated with ConfigProperty injectable - do listeners allow not having a binding but influencing how to inject the value?

Or were you suggesting writing an extension that listened to type and then added a binding - the challenge there is that you can't create a key with any annotation, which doesn't contain javax.inject.Qualifier or com.google.inject.BindingAnnotation, which ConfigProperty does not

from guice.

Yep - I got pretty well through it but very busy time work wise for me right now -

Intercepting on the injection with a TypeListener and pushing the value into the configuration seems to be bringing some success for me, support for 3rd party annotations generally goes this route to cater for these scenarios,
Whats also pretty nice is the config library can also be bound as singletons or eager singletons as well which I'm playing with

Reflection probably not going to get away from, in one way or the other, but the binding mechanism :) I need it for something i'm working on during the 9-5 as well, but have a lot in front before I get there :)

I'm doing an approach very similar to the jsr250 implementation

https://github.com/mathieucarbou/guice/blob/master/extensions/injection/src/main/java/com/mycila/guice/ext/injection/MemberInjectorTypeListener.java

bindListener(Matchers.any(), new TypeListener() {
            @Override
            public <I> void hear(TypeLiteral<I> type, TypeEncounter<I> encounter) {
               
            }
        });

from guice.

Your work is amazing and inspiring and makes me wonder - "how on earth did you figure out how to do that?" The documentation doesn't really go into much detail about extending or plugging in, and from what experimentation I've tried - I can't see how only having the TypeListener is able to do the injection and communicate to Guice internals that it's already taken care of... if I try with a simple listener and add a memberInjector and just set the value, Guice still complains. Is there some magic communication that happens when you ask the "encounter" for the Injector and then ask it for a provider? I see the key is rewritten to look for Named but how does Guice know that you've done that for it?

Ultimately, I would like to not have to prebind anything - and this mechanism looks really great if I can just get the Config object and get the value (which is easy) then inject it, it would be quite nice to use - but squelching Guice's lookup failures is the part I can't make work.

from guice.

Ohhhhhh… This doesn't work if you have @Inject AND @Resource (or some other non-injectible thing) - for some reason, I thought that you had figured out the magic to let listeners adjust the bindings, but, in fact, your clients would have to NOT declare @Inject - I'll have to think about what that means - it doesn't seem quite right… in CDI world, @ConfigProperty needs @Inject so it seems like you'd want that regardless of whether SE with Guice, or EE with CDI/Weld.

from guice.

OK that is strange I've already done this on a few of my Guiced-EE implementations and that wasn't the case - I will check it and get back to you, definitely supposed to override any @Inject settings hmm - To make sure though, this best works with PrivateModules with expose clauses - sorry about the delay the holidays in April were a few too many put quite a bit of the side-work on hold getting back into work xD

I'm going a bit overboard on the impl though, doing a main MP implementation, not just for config, let's see if I can keep them loosely coupled xD

from guice.

@GedMarc - Here's a draft PR to show you what I haven't been able to make work - but in the context of your system.

mathieucarbou/guice#20

And reference showing that the desired behavior for ConfigProfile should include @Inject

https://javadoc.io/static/org.eclipse.microprofile.config/microprofile-config-api/3.0.3/org/eclipse/microprofile/config/inject/ConfigProperty.html

 @Inject
 @ConfigProperty(name = "my.long.property")
 private Long injectedLongValue;

from guice.

Duplicate of #1383

from guice.