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

UNDERTOW-1195 correct value of FLUSH_PACKET_STRING in the mod_cluster MCMP implementation #562

Merged
merged 1 commit into from Oct 12, 2017
Merged

Conversation

frapex
Copy link
Contributor

@frapex frapex commented Oct 2, 2017

The value of FLUSH_PACKET_STRING in the mod_cluster MCMP implementation
should be set to "flushpackets", not "flushpacket".

The error can be reproduced by configuring the mod_cluster subsystem with property flush-packets set to true.

Apache httpd mod_cluster-1.3.7 accepts the node. Undertow mod_cluster implementation produces an error:
ERROR: UT005043: Error in processing MCMP commands: Type:SYNTAX, Mess: SYNTAX: Invalid field flushpackets in message

This fix should be applied to all active branches.

@undertow-pull-request
Copy link

Can one of the admins verify this patch?

@frapex
Copy link
Contributor Author

frapex commented Oct 10, 2017

@stuartwdouglas can you take a quick look at this pull request? thanks in advance!

@stuartwdouglas
Copy link
Contributor

This is ok to test

@stuartwdouglas
Copy link
Contributor

Both PR's look good to me. Would it be possible for you to file a JIRA at https://issues.jboss.org/browse/UNDERTOW and update the commit messages to reference the JIRA?

Thanks,
Stuart

@undertow-pull-request
Copy link

Windows Build 2212 outcome was FAILURE using a merge of 4695ed3
Summary: Tests passed: 2964, ignored: 613; exit code 1 Build time: 00:28:54

… MCMP implementation

The value of FLUSH_PACKET_STRING in the mod_cluster MCMP implementation
should be set to "flushpackets", not "flushpacket".
@frapex frapex changed the title correct value of FLUSH_PACKET_STRING in the mod_cluster MCMP implementation UNDERTOW-1195 correct value of FLUSH_PACKET_STRING in the mod_cluster MCMP implementation Oct 11, 2017
@frapex
Copy link
Contributor Author

frapex commented Oct 11, 2017

@stuartwdouglas JIRA issue added, commit message updated.

@undertow-pull-request
Copy link

Windows Build 2213 outcome was FAILURE using a merge of 99c33c1
Summary: Tests passed: 2964, ignored: 613; exit code 1 Build time: 00:29:18

@stuartwdouglas stuartwdouglas merged commit d6b1f70 into undertow-io:master Oct 12, 2017
@frapex frapex deleted the mod_cluster-flushpackets branch October 23, 2017 07:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants