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

Update README.md #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jean-michel-perli-sonsin

Corrected README text according to https://github.com/th2-net/th2-conn/pull/2/files


This project includes only an adapter for using the Sailfish service in the th2 packed into a Docker Image.
This image should be used as a base to implement extensions with the real logic for specific protocols using services in the Sailfish format.
This project includes only one adapter for using the Sailfish service in the th2 package into the Docker Image.
Copy link
Member

Choose a reason for hiding this comment

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

This sentence has completely changed its meaning.
This project is the adapter that is packed into the Docker Image. This adapter allows users to use the Sailfish services in the th2

Here you can automatically send heartbeat messages, send a logon/logout, requests to retransmit messages in the event of a gap, etc.
The "Connect" component uses a separate queue to send messages. The component subscribes to that pin at the start and wait for messages to be received.
The received messages from that pin will be sent to the target system.
Besides, this component is responsible for maintaining connections and sessions , in the cases provided by the communication protocol.
Copy link
Member

Choose a reason for hiding this comment

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

Redundant space before the colon

Suggested change
Besides, this component is responsible for maintaining connections and sessions , in the cases provided by the communication protocol.
Besides, this component is responsible for maintaining connections and sessions, in the cases provided by the communication protocol.

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