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.

Sign up for GitHub

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

Jump to bottom

[REM3-259] (4.0) Authentication via remoting fail for larger requests i.e. long password #113

Merged
merged 1 commit into from Jun 28, 2017
Merged

[REM3-259] (4.0) Authentication via remoting fail for larger requests i.e. long password #113

merged 1 commit into from Jun 28, 2017

Conversation

rnetuka
Copy link

@rnetuka rnetuka commented May 16, 2017

@rnetuka rnetuka mentioned this pull request May 16, 2017
Merged
dmlloyd
dmlloyd requested changes May 19, 2017
View reviewed changes
Copy link
Member

@dmlloyd dmlloyd left a comment

Choose a reason for hiding this comment

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

Like #111, this PR causes hangs in the test suite.

@rnetuka rnetuka force-pushed the large-buffer-for-password-40 branch from d9cd183 to f77e070 Compare May 23, 2017 08:10
@rnetuka
Copy link
Author

rnetuka commented May 23, 2017

There was a NullPointerException emerging when running ConnectionTestCase. I've added null check. The test should be ok now.
(force pushed the changes to my repo - please retry again)

@dmlloyd
Copy link
Member

dmlloyd commented May 23, 2017

I get 10 errors now:

Tests in error: 
  RemoteSslChannelTest.testStart:92 » IO Broken pipe
  RemoteSslChannelTest.testStart:92 » IO Broken pipe
  RemoteSslChannelTest.testStart:92 » IO Broken pipe
  RemoteSslChannelTest.testStart:92 » IO Broken pipe
  RemoteSslChannelTest.testStart:92 » IO Broken pipe
  RemoteSslChannelTest.testStart:92 » IO Broken pipe
  RemoteSslChannelTest.testStart:92 » IO Broken pipe
  RemoteSslChannelTest.testStart:92 » IO Broken pipe
  RemoteSslChannelTest.testStart:92 » IO Broken pipe
  RemoteSslChannelTest.testStart:92 » IO Broken pipe

@rnetuka
[REM3-259] Authentication via remoting fail for larger requests e.g. …
76d85d1 
…long password

[REM3-259] Additional fix for remoting 4.0.22
@rnetuka rnetuka force-pushed the large-buffer-for-password-40 branch from f77e070 to 76d85d1 Compare May 24, 2017 13:53
@rnetuka
Copy link
Author

rnetuka commented May 24, 2017

Fixed. The readMessageLength() method now returns 0 when length cannot be read (code fragment taken from FramedMessageChannel).

@dmlloyd dmlloyd merged commit 7700e13 into jboss-remoting:4.0 Jun 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dmlloyd dmlloyd dmlloyd approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@rnetuka @dmlloyd