Commit Graph

941 Commits

Author SHA1 Message Date
Ivan Kozlovic
f75779b65b Use 2 files for CloneTLSConfig instead of reflection
Use build directives to solve the problem of new fields in tls.Config
in go 1.7

Related to #231
2016-10-20 20:39:39 -06:00
Derek Collison
2c7f1fa6c0 Merge pull request #363 from nats-io/fix_tls_config_copy
[FIXED] assignment copies lock value for crypto/tls.Config
2016-10-20 14:07:08 -07:00
Ivan Kozlovic
4997637270 [FIXED] assignment copies lock value for crypto/tls.Config
Running `go vet ./...` with `go 1.7.3` would report the following:

```
server/route.go:342: assignment copies lock value to tlsConfig: crypto/tls.Config contains sync.Once contains sync.Mutex
server/server.go:479: assignment copies lock value to config: crypto/tls.Config contains sync.Once contains sync.Mutex
```

Add a “clone” function while waiting for this to be addressed
by the language itself (https://go-review.googlesource.com/#/c/28075/)
2016-10-20 14:59:29 -06:00
Derek Collison
10cd7aa16d Merge pull request #362 from nats-io/syslog-tag
Generate syslog tags based on the name of the executable or link.
2016-10-19 07:52:42 -07:00
Colin Sullivan
c3753d4d27 Generate syslog tag based on executable/link name 2016-10-18 12:50:49 -06:00
Derek Collison
8da901fcd9 Merge pull request #360 from nats-io/fix_setlogger
[FIXED] SetLogger to be able to set debug/trace to 0
2016-10-06 08:50:55 -07:00
Ivan Kozlovic
9f758bbd0d Use simple if/else 2016-10-06 09:30:49 -06:00
Ivan Kozlovic
44d18c3af6 [FIXED] SetLogger to be able set debug/trace to 0
In NATS Streaming, we have a test - started early on - that was testing
logging and called SetLogger(l, true, true), then reset by calling
SetLogger(l, false, false) to reset the values. That obviously had not
the expected effect.
During profiling, I noticed that there were tons of allocated
objects due to NATS server debug/trace statements caused by that.
2016-10-05 17:56:25 -06:00
Derek Collison
8caed5eefd Merge pull request #353 from bentranter/fix-non-ascii-quotes-in-html
Fix non-ASCII quotes in HTML
2016-09-12 23:29:46 -04:00
Ben Tranter
ad6206078f Fix non-ASCII quotes in HTML
Fixes #304

Changes the non-ASCII curly quotes in the HTML to use the regular ASCII double quotes. Also adds a test to check for the existence of non-ASCII characters in that same HTML.
2016-09-12 11:03:19 -04:00
Derek Collison
e41d360e77 Merge pull request #337 from nats-io/fix_tls_verify
[FIXED] Client certificate verification when `verify` is true.
2016-08-25 17:44:11 -05:00
Ivan Kozlovic
8f7f6e9f8b [FIXED] Client certificate verification when verify is true.
Server was incorrectly requiring a client certificate, but not
verifying it.

Resolves #336
2016-08-25 15:33:13 -06:00
Derek Collison
c7e699ac29 Updated benchmark formatting, added 128b payload 2016-08-24 13:17:07 -07:00
Ivan Kozlovic
dcbf5d8d35 Bump to version v0.9.4 v0.9.4 2016-08-18 09:27:43 -06:00
Derek Collison
21e583628f Merge pull request #333 from wallyqs/custom-monitoring-server-listen
Enables using the same listen from server for monitoring endpoint via `-a`
2016-08-18 05:33:46 -05:00
Waldemar Quevedo
e46b187230 Enables using the same listen from server as was specified via -a
in the cmd line for the monitoring server as well.
2016-08-17 23:08:37 -07:00
Derek Collison
1b8cde624a Merge pull request #330 from nats-io/ignore_any_in_returned_ips
Check that 0.0.0.0 or [::] is not returned as a client connect URL
2016-08-17 23:20:34 -05:00
Derek Collison
859949e70d Merge pull request #332 from nats-io/fix_race_on_unsub
[FIXED] Data RACE on Unsubscribe when client connection is closed
2016-08-17 23:04:33 -05:00
Ivan Kozlovic
811e0868ed [FIXED] Data RACE on Unsubscribe when client connection is closed
Resolves #331
2016-08-17 16:46:34 -06:00
Ivan Kozlovic
c543c77c74 Check that 0.0.0.0 or [::] is not returned as a client connect URL 2016-08-17 11:15:05 -06:00
Derek Collison
14f5d0919c Merge pull request #328 from nats-io/fix_concurrent_monitor_poll
[FIXED] Server panic when poll for Varz and others concurrently
2016-08-17 06:31:02 +02:00
Ivan Kozlovic
e6039e0a8b [FIXED] Server panic when poll for Varz and others concurrently
Resolves #327
2016-08-16 10:50:49 -06:00
Derek Collison
638e249d2a Merge pull request #326 from nats-io/disable_async_info
[ADDED] Option to not advertise to clients cluster's IPs
2016-08-16 02:27:42 +02:00
Ivan Kozlovic
82dbb3a5ab [ADDED] Option to not advertise to clients cluster's IPs
By default, a server is now sending to its clients the client URLs
of all servers in the cluster. This allows clients to be able
to reconnect to any server in the cluster even if those clients
were not configured with the list of servers in the cluster.

However, there may be cases where it would make sense to disable
this feature. This now can be done with this option/command line
parameter.

Resolves #322
2016-08-12 19:24:12 -06:00
Derek Collison
a31e6b2b77 Merge pull request #324 from nats-io/fix_route_ipv6
[FIXED] Cluster's listener with IPv6
2016-08-12 15:20:36 -07:00
Derek Collison
29b31010cc Merge pull request #325 from nats-io/fix_link_local
[FIXED] When server listens to any interface, return only global IPs
2016-08-12 15:19:43 -07:00
Ivan Kozlovic
3b8412049e [FIXED] Cluster's listener with IPv6
Trying to use IPv6 address for the cluster host would fail.
Also, there were some unclosed channels in case of accept loop
setup failures.

Resolves #323
2016-08-12 15:54:15 -06:00
Ivan Kozlovic
fc7d92a04a [FIXED] When server listens to any interface, return only global IPs
The server was returning all resolved IP addresses, including link
local addresses, which did not make sense for remote clients.

Resolves #321
2016-08-12 14:22:24 -06:00
Ivan Kozlovic
62923be357 Bump version to 0.9.2 v0.9.2 2016-08-08 14:56:28 -06:00
Derek Collison
5e68e38c8b Merge pull request #319 from nats-io/fix_cluster_override
[FIXED] Route/Cluster override
2016-08-04 18:20:50 -06:00
Ivan Kozlovic
5185f10fed [FIXED] Route/Cluster override
If the server was started with a cluster section in a configuration
file and one would want to override the routes (using `-routes`) the
server would complain that you need to use `-cluster`. Adding
an override of cluster would not work, server would still complain.
Trying to override simply the cluster listen info (without override
of routes) would also not work.
2016-08-04 13:20:25 -06:00
Derek Collison
a2c747a88d Merge pull request #318 from nats-io/docker_update
[UPDATED] Dockerfile to use go 1.6.3 [ci skip]
2016-08-03 11:11:40 -06:00
Ivan Kozlovic
f6cb458480 [UPDATED] Dockerfile to use go 1.6.3 [ci skip] 2016-08-03 09:29:28 -06:00
Derek Collison
77a06b3b16 Merge pull request #316 from nats-io/improve_route_perf_big_msgs
[IMPROVED] Route performance for larger messages
2016-08-01 18:06:48 -07:00
Ivan Kozlovic
c6d37ecf8f [IMPROVED] Route performance for larger messages 2016-08-01 19:01:34 -06:00
Derek Collison
e0bb79865c Merge pull request #314 from nats-io/server_send_async_info
[ADDED] Server sends INFO with cluster URLs to clients with support
2016-08-01 16:53:50 -04:00
Ivan Kozlovic
6f9d542393 Minor changes based on comments 2016-07-26 12:25:33 -06:00
Ivan Kozlovic
61b0fec200 Updated test for corner cases
Test when client has not yet sent its CONNECT proto or the first
PING when route is added.
2016-07-26 11:57:16 -06:00
Ivan Kozlovic
fda5bd7ac7 [ADDED] Server sends INFO with cluster URLs to clients with support
Clients that will be at the ClientProtoInfo protocol level (or above)
will now receive an asynchronous INFO protocol when the server
they connect to adds a *new* route. This means that when the cluster
adds a new server, all clients in the cluster should now be notified
of this new addition.
2016-07-26 10:55:55 -06:00
Derek Collison
1ac073d49d Merge pull request #313 from nats-io/fix-typo
Fix tls parameter typo
2016-07-25 08:23:01 -07:00
Colin Sullivan
ab962bf0dd Fix typo in tls parameter doc and readme. 2016-07-25 08:50:29 -06:00
Derek Collison
9a19e5a6c6 update broken coveralls badge image 2016-07-20 13:39:05 -07:00
Derek Collison
d95765f052 Merge pull request #311 from nats-io/fix_pse_test
[FIXED] PS test to reduce failures in some situations
2016-07-18 14:12:29 -05:00
Derek Collison
fa4d3da26a Bump travis to use new version of 1.6 2016-07-18 12:07:34 -07:00
Ivan Kozlovic
d21210e0e9 [FIXED] PS test to reduce failures in some situations
Added garbage collection before each capture method and bump the
delta threshold to 1MB.

Resolves #310
2016-07-18 13:00:49 -06:00
Derek Collison
b14d3b9405 Merge pull request #308 from nats-io/readme-clustering
Updates to Clustering section
2016-07-16 01:36:18 -05:00
Alberto Ricart
d022275087 [ci skip] Changes by Ivan and Alberto 2016-07-15 15:58:34 -05:00
Alberto Ricart
92b5a32586 Updated clustering description to show that full mesh is required. 2016-07-15 15:11:22 -05:00
Derek Collison
c58af70095 Merge pull request #309 from nats-io/control_overflow
Control overflow
2016-07-12 02:38:01 +02:00
Derek Collison
ab11e04bf9 Fixes for gofmt -s 2016-07-11 12:16:21 -07:00