AnnotationIntrospector doesn't override JsonAlias

Hello all!

Thank you for the awesome work. While using Jackson within my own framework, I came across some potentially weird behaviour.

Use Case: I want to dynamically add Json Aliases to fields marked with a certain annotation(that I don’t control)

I would expect that overriding like this should do the trick, but it doesn’t. If I add the JsonAlias annotation myself, it works.

@Override
    public List<PropertyName> findPropertyAliases(Annotated ann) {
        if (isId(ann)) {
            return List.of(new PropertyName("_id"), new PropertyName("id"));
        }
        return super.findPropertyAliases(ann);
    }
    
    

    private boolean isId(Annotated annotated) {

        return annotated.hasAnnotation(Id.class)
                || annotated.hasAnnotation(javax.persistence.Id.class)
                || annotated.hasAnnotation(Identifier.class)
                || annotated.getName().equals("id");

    }

[cowtowncoder]

@RyanSusana that is weird. Thank you for investigating this part. Code looks legit. About the only thing better than this would be unit test based on pieces above – I’ll add this to my todo list to hopefully check out soon (depending on other tasks). Or, if you have time and interest, you can create one and attach too: just needs to have no additional dependencies so can reduce to minimal reproduction (just check for id named property for example, pass in “_id”).

One other thing: make sure to test with latest Jackson version (2.10.3) if not using that – there is at least one fix (#2378 in 2.10.0) since addition of @JsonAlias).

[cowtowncoder]

@ryangardner Excellent code spelunking!