Steve Riesenberg
Steve Riesenberg
Thanks @mikesaurus, very helpful information/feedback to ensure we're working towards something the community can use. You're in a perfect position to give that feedback because you're porting from the old...
@bjornharvold, that's great! Feel free to create a separate github project for it. We won't be including a mongo (or redis) implementation in the core, though what we're discussing here...
For additional context on this issue, see [this comment](https://github.com/spring-projects/spring-security/issues/11162#issue-1214211098) by @Kehrlann and this [sample repo](https://github.com/Kehrlann/spring-security-issue-securityfilterchain-authenticationeventpublisher) that demonstrates the issue. I think we could make the upgrade path to using an...
Fixed via 0aac515737e385feca81286a6f0bfd1ef2bb90ab (duplicate of gh-11449).
@shazin, just checking to see if you saw the comment above and are able to update this PR to remove `existingAuth instanceof UsernamePasswordAuthenticationToken` as suggested in [@rwinch's comment](https://github.com/spring-projects/spring-security/issues/10347#issuecomment-1147754004)?
Merged via 1e0e9a2c98d6f37aa8989801a75fb434cd953c06 along with e0e6467d9b1203b38d152b40281265e0816af8be
Thanks @ch4mpy! As we're working through some priority tasks for 5.8 and 6.0, it may be a slight delay before I'm able to take a look at this.
Thanks for taking the time! We use the duplicate label for book-keeping so only one issue (either the issue or the PR) goes into the release notes, and so whoever...
Thanks @ch4mpy. Apologies, I haven't been able to review it thoroughly yet, as I'm still heads down. However, glancing at it, I wanted to mention one thing to think about....
> Please confirm that IntrospectionResult is fine for naming the converter source type. I'll spend some time thinking about that.