Skip to content

fix bugs#138

Closed
arden wants to merge 17 commits intomasterfrom
develop
Closed

fix bugs#138
arden wants to merge 17 commits intomasterfrom
develop

Conversation

@arden
Copy link

@arden arden commented Jul 23, 2019

fix bugs

tomponline and others added 17 commits July 21, 2018 10:27
This commit improves support for NATS 1.2.x by checking if several
fields; `auth_required`, `tls_required`, and `tls_verify` are set in
the initial INFO packet. As of NATS 1.2, these fields aren't always
set, which can cause an E_NOTICE to be raised.

Without this change, the unit tests are failing because the Docker
container has been updated to 1.2.0, and PHPUnit is configured to
treat notices as test failures.

See: nats-io/nats-server@17fecd4#diff-91bbeda7eb98a7adc57b9e47e2cf5c2b
@arden
Copy link
Author

arden commented Jul 23, 2019

fix bugs.

@arden arden closed this Jul 23, 2019
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