Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

APP-3020: DatafeedService retry with the SocketTimeoutException #209

Merged
merged 11 commits into from
Sep 11, 2020

Conversation

symphony-hong
Copy link
Contributor

No description provided.

Create ServiceFactory class for providing services to SymphonyBdk entry point.

Simplify code in SymphonyBdk

Experimental: First implementing OBO call on StreamService.

User can do an OBO authentication by calling SymphonyBdk#obo(Obo.Handle). This method will return an AuthSession which is an obo session.

OboStreamService contains only OBO-enabled stream apis. User will do the OBO authentication out of scope of SymphonyBdk, then will pass the AuthSession which is returned by the previous step to the method to perform the call with oboSession.
Create OboUserService for calling user service methods using OBO Session

Remove methods wrapping the deprecated user service API

Remove redundant OBO class
Copy link
Contributor

@symphony-elias symphony-elias left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@symphony-hong symphony-hong merged commit 2343feb into finos:master Sep 11, 2020
@symphony-hong symphony-hong deleted the APP-3020 branch September 11, 2020 16:20
symphony-elias added a commit that referenced this pull request Sep 11, 2020
* APP-3020: DatafeedService retry with the SocketTimeoutException (#209)

* Fluent API setup

* Bumped version to 1.2.0 for legacy and 1.2.1.BETA for BDK2.0

Co-authored-by: symphony-hong <65538951+symphony-hong@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants