-
Notifications
You must be signed in to change notification settings - Fork 121
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
Allowing application to request logoutDisconnect in a session and spe… #488
Allowing application to request logoutDisconnect in a session and spe… #488
Conversation
@@ -267,9 +267,13 @@ | |||
return string != null && string.length() > 0; | |||
} | |||
|
|||
public long sendLogout(final int msgSeqNo, final int sequenceIndex, final int lastMsgSeqNumProcessed) | |||
public long sendLogout( |
Check notice
Code scanning / CodeQL
Missing Override annotation Note
SessionProxy.sendLogout
@@ -408,13 +408,18 @@ | |||
return acceptingSessionWriter.send(logon, adjustedMsgSeqNo); | |||
} | |||
|
|||
public long sendLogout(final int msgSeqNo, final int sequenceIndex, final int lastMsgSeqNumProcessed) | |||
public long sendLogout( |
Check notice
Code scanning / CodeQL
Missing Override annotation Note test
SessionProxy.sendLogout
9e162d7
to
b4f3015
Compare
artio-core/src/main/java/uk/co/real_logic/artio/session/Session.java
Outdated
Show resolved
Hide resolved
artio-system-tests/src/test/java/uk/co/real_logic/artio/system_tests/SystemTestUtil.java
Outdated
Show resolved
Hide resolved
…cify a custom text
b4f3015
to
30ada28
Compare
i've just pushed again implementing your suggestions - thx |
…cify a custom text (real-logic#488)
…cify a custom text