DispatcherHandler no longer orders HandlerMapping beans [SPR-15991]

[spring-projects-issues]

Andy Wilkinson opened SPR-15991 and commented

This change has introduced a regression which means that HandlerMapping beans are no longer ordered. Currently, this.handlerMappings is being sorted:

ArrayList<HandlerMapping> mappings = new ArrayList<>(mappingBeans.values());
AnnotationAwareOrderComparator.sort(this.handlerMappings);
this.handlerMappings = Collections.unmodifiableList(mappings);

I believe it should be mappings that is sorted:

ArrayList<HandlerMapping> mappings = new ArrayList<>(mappingBeans.values());
AnnotationAwareOrderComparator.sort(mappings);
this.handlerMappings = Collections.unmodifiableList(mappings);

---

Affects: 5.0 GA

Issue Links:

• Provide an API for introspection of handler mappings [SPR-15934] #20488 Provide an API for introspection of handler mappings

Referenced from: commits 816a58f, 31619b3, f9b319d

[spring-projects-issues]

Rossen Stoyanchev commented

Should be fixed now, I've also added a test ;).