Support Determining Max Sessions by Authentication

[ClaudenirFreitas]

Closes gh-16206

[pivotal-cla]

@ClaudenirFreitas Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

[pivotal-cla]

@ClaudenirFreitas Thank you for signing the Contributor License Agreement!

[jzheaux]

Thanks for such a quick turnaround, @ClaudenirFreitas! I've left my feedback inline.

[ClaudenirFreitas]

Thanks for such a quick turnaround, @ClaudenirFreitas! I've left my feedback inline.

thanks a lot, @jzheaux.
I just pushed some changes and also replied to some comments. Could you review it again?

Meanwhile, I am still working on creating some integration tests to cover the configurations via XML, ok?

[jzheaux]

Thanks for the updates, @ClaudenirFreitas! I've left some additional feedback inline.

Let me know when you have integration tests ready, and I'll take another look.

[ClaudenirFreitas]

Thanks for the updates, @ClaudenirFreitas! I've left some additional feedback inline.

Let me know when you have integration tests ready, and I'll take another look.

@jzheaux integration tests have been added.

Many tests failed due to the new release (6.5). I've updated many files on this commit, and the tests are passing now.

Silly question, is this flow correct for the new release?

[ClaudenirFreitas]

Thanks for the updates, @ClaudenirFreitas! I've left some additional feedback inline.
Let me know when you have integration tests ready, and I'll take another look.

@jzheaux integration tests have been added.

Many tests failed due to the new release (6.5). I've updated many files on this commit, and the tests are passing now.

Silly question, is this flow correct for the new release?

Thanks for the updates, @ClaudenirFreitas! I've left some additional feedback inline.
Let me know when you have integration tests ready, and I'll take another look.

@jzheaux integration tests have been added.

Many tests failed due to the new release (6.5). I've updated many files on this commit, and the tests are passing now.

Silly question, is this flow correct for the new release?

@jzheaux
I just rebased this PR due to the current main changes.
I also added the unit and integration tests.
Could you review them again?

[jzheaux]

Just the finishing touches now! Please see my inline feedback.

Also, please squash your commits at this point and format it like so:

Add Max Sessions by Authentication

Closes gh-16206

[ClaudenirFreitas]

Just the finishing touches now! Please see my inline feedback.

Also, please squash your commits at this point and format it like so:

Add Max Sessions by Authentication

Closes gh-16206

@jzheaux
I had some conflicts when squashing the commits. Could use the Squash and merge option? e.g.

[jzheaux]

Thanks for asking. We don't prefer to add a merge commit to the history in this case. Do you need help squashing?

[ClaudenirFreitas]

Thanks for asking. We don't prefer to add a merge commit to the history in this case. Do you need help squashing?

@jzheaux Yes, please.

[jzheaux]

Thanks for all your work on this PR, @ClaudenirFreitas! This is now merged into main. I also added a polish in 1104b45 to add Kotlin support, adjust SessionLimit's package, and add documentation.

[ClaudenirFreitas]

Thanks for all your work on this PR, @ClaudenirFreitas! This is now merged into main. I also added a polish in 1104b45 to add Kotlin support, adjust SessionLimit's package, and add documentation.

Thanks a lot for the help, @jzheaux!

We could add a sample using this config in the spring-security-samples repo, what do you think?