Resolve package tangle between org.springframework.security.config.web and org.springframework.security.config.annotation.web.

Comment From: nor-ek

Previous thought:

Hey, what's the direction of the changes? Is moving .server into the annotation package enough? Could you let me know where to find more information? I'm new to that module. Despite the above, my thought. I suppose SecurityWebFiltersOrder may be replaced by FilterOrderRegistration. The goal of the task is to ensure that annotation contains replacements for the previous package?

Edit: After deeper research I've noticed that you are probably talking about Kotlin package org.springframework.security.config.web.servlet. As I see many Dsl are using java classes that are in org.springframework.security.config.annotation.web. It's not expected behavior, yep? Could you tell me then what's the direction of expected changes is?

Comment From: jzheaux

Hi, @nor-ek, yes you are correct that this has to do with the classes in src/main/kotlin. I believe these can simply be moved to the org.springframework.security.config.annotation.web package, still in src/main/kotlin.

As this is a breaking change, let's wait until work on 6.x is started so this can be done on that branch.

Comment From: nor-ek

@jzheaux Okay, so the task is to move all Dsl files from org.springframework.security.config.web.servlet that have dependencies to org.springframework.security.config.annotation.web?

You can assign me to that task. I'll do this on the branch and that will wait till 6.x. I'm open to other first-timers tasks if you have any in mind.

Comment From: jzheaux

@nor-ek, the 6.0.x branch is now available. Thanks for taking a look.

Comment From: nor-ek

@jzheaux Pull-request is available here: https://github.com/spring-projects/spring-security/pull/10429.

Finally I've moved files to org.springframework.security.config.annotation.web.configurers, more info in PR. Let me know if that's fine.

Comment From: nor-ek

@jzheaux sorry for the mess above. Finally, I've got Pull Request.