Revelation Builder constructor

[leshalv]

No description provided.

[leshalv]

Deserialize and assemble OAuth2Authorization in a user-defined mode

[jgrandja]

@leshalv

The Builder is not intended to be directly instantiated. The factory methods withRegisteredClient() and from() should be used instead.

Deserialize and assemble OAuth2Authorization in a user-defined mode

I don't quite understand this statement. Please provide more specific details.

[leshalv]

When customizing redis storage, deserialization is required. RegisteredClient is required to build an OAuth2Authorization object, which requires encapsulation of RegisteredClient. In fact, when building OAuth2Authorization, only registeredClientId is enough.

[jgrandja]

@leshalv OAuth2Authorization and RegisteredClient are core domain classes within the framework and are not intended to represent an Entity or DTO, that can more easily be serialized. If you're looking to serialize these objects into a Redis store then you need to create an Entity/DTO representation of those objects so it can be serialized.

There is a discussion and some examples provided in gh-558.

We are also planning on providing a How-to guide in gh-1019 to demonstrate how to implement it. You can also look at the existing How-to: Implement core services with JPA to see how the Entity representations can look like and instead of persisting to a sql db, you can serialize to JSON for Redis storage.

I'm going to close this PR as we don't want to expose the Builder constructor as the factory methods should be used instead.