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

Add Reactor Netty version of secure chat server example reactor#761 #2677

Merged
merged 1 commit into from
Feb 7, 2023

Conversation

jchenga
Copy link
Contributor

@jchenga jchenga commented Jan 29, 2023

Provide a corresponding Reactor Netty version of Secure chat server example as found in Netty project.

Related to #761

@jchenga jchenga force-pushed the 761-secure-chat-server branch 2 times, most recently from a18e566 to e3760b6 Compare January 29, 2023 19:47
@jchenga
Copy link
Contributor Author

jchenga commented Jan 30, 2023

I am not sure why build (windows-2019, nio) failed. the check failed with assertion exception at ITTracingHttpClientDecoratorTest, but the same test succeeded with other OS platforms.

@pderop pderop self-assigned this Jan 31, 2023
Copy link
Contributor

@pderop pderop left a comment

Choose a reason for hiding this comment

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

Hi @jchenga ,

thanks a lot for submitting this PR !
Now, I have made few comments, can you please check the following ?

@pderop pderop added this to the 1.0.28 milestone Feb 1, 2023
@pderop pderop added the type/enhancement A general enhancement label Feb 1, 2023
@violetagg violetagg added type/documentation A documentation update and removed type/enhancement A general enhancement labels Feb 1, 2023
@violetagg violetagg modified the milestones: 1.0.28, 1.1.3 Feb 1, 2023
Copy link
Contributor

@pderop pderop left a comment

Choose a reason for hiding this comment

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

thanks @jchenga,

+1

@violetagg , please take a look

@pderop pderop requested a review from violetagg February 2, 2023 08:11
Copy link
Member

@violetagg violetagg left a comment

Choose a reason for hiding this comment

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

nice!

Provide a corresponding Reactor Netty version of Secure chat server example as found
in Netty project.
@pderop pderop merged commit cb6e891 into reactor:main Feb 7, 2023
pderop added a commit that referenced this pull request Feb 7, 2023
@pderop
Copy link
Contributor

pderop commented Feb 7, 2023

@jchenga , thanks for the PR;
@violetagg , thanks for the review; I have merged it into main and netty5 branches.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/documentation A documentation update
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants