Madhura Bhave

Results 37 comments of Madhura Bhave

Thanks for the input, @vpavic. We discussed the option to provide `getSession` and `deleteSession` for non-indexed repositories on our last team call and decided we would leave things as they...

Placing this `on-hold` because we've decided to approach this on a case-by-case basis for new usages. There are subtle differences between `List.of()` and the equivalent `Collections` method such as `contains(null)`...

@dreis2211 We discussed this on the team call today and decided that `Collections.emptyList()` was more explicit than `List.of()` for creating empty lists. However, we might decide to change this in...

Placing on hold for the same reason as #28174.

Thanks @jsattler for pointing that out. We have decided not to introduce the technology split as it could inflict unwanted upgrade pain for users without too much benefit. We've decided...

I'd initially thought adding a `JwtDecoderCustomizer` would work but that's not the case. We would need a `JwtDecoderBuilderCustomizer` to support customizations. The problem is that the `JwtDecoder` built when using...

Yeah, the public key version doesn't need a restOperations. The `issuerUri` one, however, does use one but it doesn't appear to be customizable. Given that, I don't think we should...

Thanks for the proposal @m-kay. It is true that for the customizations that Spring Security currently supports, it is only the `JwkSetUriJwtDecoderBuilder` that needs a customizer. However, creating a `JwkSetUriJwtDecoderBuilderCustomizer`...

Placing on hold for the same reason as #28174

`spring-cloud-starter-consul-config` appears to be broken as well so maybe it's a broader problem. Projects generated with `spring-cloud-starter-vault-config` aren't broken as we explicitly mark that one as [not a starter](https://github.com/spring-io/start.spring.io/blob/3fd8e39d730070f0b4548f15efeaaca804dbe312/start-site/src/main/resources/application.yml#L909).